Skip to content

Updated documentation about the usage of CodeInfo#2276

Merged
sphuber merged 8 commits into
aiidateam:provenance_redesignfrom
zhubonan:issue_1000_add_note_about_CalcInfo
Dec 6, 2018
Merged

Updated documentation about the usage of CodeInfo#2276
sphuber merged 8 commits into
aiidateam:provenance_redesignfrom
zhubonan:issue_1000_add_note_about_CalcInfo

Conversation

@zhubonan
Copy link
Copy Markdown
Contributor

@zhubonan zhubonan commented Dec 3, 2018

Fixes #1000

Added notes in the plugin development documentation that multiple Code and hence CodeInfos may be used for a single JobCalculation to allow pre/post-processing or packing multiple jobs in a single submission if necessary.
We should be able to close issue #1000 now.

Both CalcInfo and CodeInfo are disscussed already in the two examples
(QE and integer summation). Note about the support of multiple codes for a
single calculation is added. This feature was not clearly documented eariler.
@zhubonan zhubonan requested a review from ltalirz December 3, 2018 17:57
@ltalirz
Copy link
Copy Markdown
Member

ltalirz commented Dec 3, 2018

@giovannipizzi Can you have a look at this?
Should we add some hint on what use cases we envisage for this feature?

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 3, 2018

Coverage Status

Coverage decreased (-0.03%) to 68.979% when pulling 9e58ead on zhubonan:issue_1000_add_note_about_CalcInfo into 85d7309 on aiidateam:provenance_redesign.

@giovannipizzi
Copy link
Copy Markdown
Member

As discussed, let's merge with the QE input tutorial

@zhubonan
Copy link
Copy Markdown
Contributor Author

zhubonan commented Dec 4, 2018

OK I will remove the QE example and add anything not already mentioned to the interger summation tutorial.

@sphuber sphuber merged commit 9891508 into aiidateam:provenance_redesign Dec 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants