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.

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

📚 Update PwBaseWorkChain example #18

Merged
merged 3 commits into from
May 7, 2024
Merged

📚 Update PwBaseWorkChain example #18

merged 3 commits into from
May 7, 2024

Conversation

mbercx
Copy link
Member

@mbercx mbercx commented Jan 19, 2024

Fixes #17

@mbercx
Copy link
Member Author

mbercx commented Jan 19, 2024

@DrFedro I can't add you as a reviewer since you're not a collaborator on the repo, but please have a look and let me know if the example is clear and if you have any comments?

@mbercx
Copy link
Member Author

mbercx commented Jan 19, 2024

Note: I'm also still updating the addition table example.

@mbercx mbercx force-pushed the imp/pw-base-example branch 2 times, most recently from 9d341b7 to e04b8b9 Compare January 19, 2024 12:06
@DrFedro
Copy link

DrFedro commented Apr 24, 2024

@DrFedro I can't add you as a reviewer since you're not a collaborator on the repo, but please have a look and let me know if the example is clear and if you have any comments?

It's all good and clear! Thanks @mbercx
On a side note, while running aiida-submission-controller with aiida-core v2.5.1, I encountered two warnings related to deprecated AiiDA methods: orm.Group.objects and node.set_extra_many().

@mbercx
Copy link
Member Author

mbercx commented Apr 24, 2024

Thanks @DrFedro! So far I hadn't updated these to also keep support for AiiDA v1, but I should probably do a final release for that and then update the deprecated methods.

mbercx added 3 commits May 7, 2024 15:21
The purpose of the `unique_extra_keys` field is that the user can define the keys of
the extras that track which calculations have been run upon instantiation of the
submission controller. However, it's also possible the user might override the
`get_extra_unique_keys()` method to fix these keys, as is done in the `add_in_batches`
example. In this case the user should not have to specify the `unique_extra_keys`,
but that currently doesn't work because we didn't set a default. This is fixed here.
@mbercx mbercx force-pushed the imp/pw-base-example branch from e04b8b9 to 6f52a34 Compare May 7, 2024 13:21
@mbercx mbercx merged commit f3f6dac into main May 7, 2024
2 checks passed
@mbercx mbercx deleted the imp/pw-base-example branch May 7, 2024 13:26
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.

🐛 BUG: Examples don't seem up to date
2 participants