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

Add I/O support for Silo Materials #1171

Merged
merged 71 commits into from
Oct 31, 2023
Merged

Add I/O support for Silo Materials #1171

merged 71 commits into from
Oct 31, 2023

Conversation

JustinPrivitera
Copy link
Member

No description provided.

@JustinPrivitera
initial work on write support
d5cb31d
@JustinPrivitera
what have i done
f217111
@JustinPrivitera
oops sorry
b3d17ed
@JustinPrivitera
to_silo fix
10f27f3
@JustinPrivitera
oops
4b389eb
@JustinPrivitera
refactor for clarity in reading top level
bb1c2ff
@JustinPrivitera
read multimat function
aeb7a74
@JustinPrivitera
fix some typos
47736f1
@JustinPrivitera
the refactor passes all tests
4958b2b
@JustinPrivitera
refactor of ovltop logic
9740eed
@JustinPrivitera
now mesh domain uses same logic as var domain
c80e6ab
@JustinPrivitera
more refactor
5a06f51
@JustinPrivitera
have the tests back
48d794f
@JustinPrivitera
ok so the map was a bad idea
54b3a51
@JustinPrivitera
material read
d9f4ed2
@JustinPrivitera
a few tests are passing
3f594e4
@JustinPrivitera
material tests and fixes
002c21a
@JustinPrivitera
confirm that materials are correct
ef8a632
@JustinPrivitera
tests
bd4b694
@JustinPrivitera
switch order of material and field read
a572416
@JustinPrivitera
same change for writing
6e4e276
@JustinPrivitera
save out field mixvals
bdf247f
@JustinPrivitera
closer to diff
08c209d
@JustinPrivitera
sort matids before saving out
6d72802
@JustinPrivitera
whoops
61c48bf
@JustinPrivitera
we're bacl
8b4a9f3
@JustinPrivitera
this is a waking nightmare
e40780a
@JustinPrivitera
fixing bugs
649f141
@JustinPrivitera
new compaction method and detail namespace
0025003
@JustinPrivitera
move detail and compile
3b02556
@JustinPrivitera
I went to town
c8fec28
@JustinPrivitera
fixed more todos
02f23b2
@JustinPrivitera
whoops
0857c38
@JustinPrivitera
bp reader in visit is changing so I make changes here too
88d6541
@JustinPrivitera
repair the damage
9eecb9f
@JustinPrivitera
Merge branch 'develop' into task/JustinPrivitera/07_19_23/silo_io_mat…
7c0bdc2 
…erials_support
@JustinPrivitera
remove bad test
ba92a77
@JustinPrivitera
remvoe old baseline
840d089
@JustinPrivitera
clean up matset transforn
4d7d303
@JustinPrivitera
wacky matnos venn test
f9d1b34
@JustinPrivitera
round trip multidom spiral with matset
2e5b64b
@JustinPrivitera
reenable tests
3a3bdd2
@JustinPrivitera
cyrus helped me
acbccfc
@JustinPrivitera
completing and adding abck in tests
fd40b27
@JustinPrivitera
verified that all normal silo files have their materials propagate co…
6a62d63 
…rrectly through tests
@JustinPrivitera
due diligence on fake overlink files
c0079e7
@JustinPrivitera
did due diligence of overlink symlink materials
3a0b7da
@JustinPrivitera
all done with serial tests
b3a8c18
@JustinPrivitera
be careful about ints
72b0436
@JustinPrivitera
added mpi tests
b891a75
@JustinPrivitera
Merge branch 'develop' into task/JustinPrivitera/07_19_23/silo_io_mat…
3253e36 
…erials_support
@JustinPrivitera
oops
eda0cb1
@JustinPrivitera JustinPrivitera mentioned this pull request Sep 18, 2023
Open
21 tasks
@JustinPrivitera JustinPrivitera mentioned this pull request Oct 3, 2023
Closed
cyrush
cyrush approved these changes Oct 31, 2023
View reviewed changes
Copy link
Member

@cyrush cyrush left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests look, lets get this in!

@JustinPrivitera JustinPrivitera merged commit d84e9f6 into develop Oct 31, 2023
21 checks passed
@JustinPrivitera JustinPrivitera deleted the task/JustinPrivitera/07_19_23/silo_io_materials_support branch October 31, 2023 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cyrush cyrush cyrush approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JustinPrivitera @cyrush