Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Change schema to allow for id or seq_id #2342

Closed
cwschilly opened this issue Sep 5, 2024 · 0 comments · Fixed by #2343
Closed

Change schema to allow for id or seq_id #2342

cwschilly opened this issue Sep 5, 2024 · 0 comments · Fixed by #2343
Assignees
@cwschilly
Labels
type: task

Comments

@cwschilly
Copy link
Contributor

cwschilly commented Sep 5, 2024 •
edited
Loading

What Needs to be Done?

Motivated by #2201 testing

Change the schema to allow either id or seq_id in the JSON. id should be assumed bit-packed, while seq_id is assumed not bit-packed.
@cwschilly cwschilly self-assigned this Sep 5, 2024
cwschilly added a commit that referenced this issue Sep 5, 2024
@cwschilly
#2342: update schema to support seq_id in tasks field when id is not …
0bd3b60 
…bit-encoded; add unit test
@cwschilly cwschilly mentioned this issue Sep 5, 2024
Merged
@cwschilly cwschilly linked a pull request Sep 5, 2024 that will close this issue
#2342: Change schema to allow for id or seq_id #2343
Merged
cwschilly added a commit that referenced this issue Sep 5, 2024
@cwschilly
#2342: wip: add logic for supporting or creating encoded ids in comm …
e8f7e17 
…field
cwschilly added a commit that referenced this issue Sep 5, 2024
@cwschilly
#2342: reduce duplication and improve bool naming
184ac8e
cwschilly added a commit that referenced this issue Sep 5, 2024
@cwschilly
#2342: fix scope error
98eb70c
cwschilly added a commit that referenced this issue Sep 6, 2024
@cwschilly
#2342: add communications to LBDataHolder test
c8fc9e6
cwschilly added a commit that referenced this issue Sep 6, 2024
@cwschilly
#2342: update schema to support seq_id in tasks field when id is not …
c849bdb 
…bit-encoded; add unit test
cwschilly added a commit that referenced this issue Sep 6, 2024
@cwschilly
#2342: wip: add logic for supporting or creating encoded ids in comm …
8138dec 
…field
cwschilly added a commit that referenced this issue Sep 6, 2024
@cwschilly
#2342: reduce duplication and improve bool naming
c36b530
cwschilly added a commit that referenced this issue Sep 6, 2024
@cwschilly
#2342: fix scope error
9386580
cwschilly added a commit that referenced this issue Sep 6, 2024
@cwschilly
#2342: add communications to LBDataHolder test
a9ec85f
cwschilly added a commit that referenced this issue Sep 6, 2024
@cwschilly
#2342: remove todo comment
dc4903b
cwschilly added a commit that referenced this issue Sep 6, 2024
@cwschilly
#2342: resolve warnings
530ff58
cwschilly added a commit that referenced this issue Sep 6, 2024
@cwschilly
#2342: fix id_key declaration in validator script
4741dc8
cwschilly added a commit that referenced this issue Sep 9, 2024
@cwschilly
#2342: simplify logic by creating elms directly from encoded ids
411ccdb
cwschilly added a commit that referenced this issue Sep 9, 2024
@cwschilly
#2342: initialize this_node_ with vt::uninitialized_destination
5381a4c
nlslatt added a commit that referenced this issue Sep 9, 2024
@nlslatt
Merge pull request #2343 from DARMA-tasking/2342-change-schema-to-all…
7045f0a 
…ow-for-id-or-seq_id

#2342: Change schema to allow for `id` or `seq_id`
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: update schema to support seq_id in tasks field when id is not …
cc0e702 
…bit-encoded; add unit test
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: wip: add logic for supporting or creating encoded ids in comm …
efd74ee 
…field
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: reduce duplication and improve bool naming
7a53331
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: fix scope error
27646fe
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: add communications to LBDataHolder test
e2a7eda
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: remove todo comment
446c032
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: resolve warnings
62e40f2
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: fix id_key declaration in validator script
3b544c0
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: simplify logic by creating elms directly from encoded ids
889efa4
cwschilly added a commit that referenced this issue Sep 20, 2024
@cwschilly
#2342: initialize this_node_ with vt::uninitialized_destination
c940f1f
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: update schema to support seq_id in tasks field when id is not …
69df764 
…bit-encoded; add unit test
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: wip: add logic for supporting or creating encoded ids in comm …
6b52e07 
…field
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: reduce duplication and improve bool naming
eb498dd
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: fix scope error
7e560b3
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: add communications to LBDataHolder test
dc3277a
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: remove todo comment
56ec11f
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: resolve warnings
387f534
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: fix id_key declaration in validator script
c1eb86e
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: simplify logic by creating elms directly from encoded ids
7a5f7cc
cz4rs pushed a commit that referenced this issue Sep 20, 2024
@cwschilly @cz4rs
#2342: initialize this_node_ with vt::uninitialized_destination
f77e712
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cwschilly cwschilly

Labels
type: task
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#2342: Change schema to allow for id or seq_id DARMA-tasking/vt
1 participant
@cwschilly