Change MatPesGGAPlusMetaGGAStaticMaker.output
to dict containing both statics
#586
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change
MatPesGGAPlusMetaGGAStaticMaker.output
fromstatic2.output
to{"static1": static1.output, "static2": static2.output}
. Brings it more in line with e.g.BandStructureMaker
:atomate2/src/atomate2/vasp/flows/core.py
Lines 143 to 156 in 5aced19
ac67d33 pytest addopts = "-p no:warnings --import-mode=importlib"
4a7a942 fix should_stop_children RuntimeError err msg calling handle_unsuccessful defuse_unsuccessful
7f6219a MatPesGGAPlusMetaGGAStaticMaker change output from static2.output to {"static1": static1.output, "static2": static2.output}
e47db54 test_matpes_gga_plus_meta_gga_static_maker assert isinstance(flow.output, dict)
5aced19 rename output to task_doc in matpes and mp tests