-
Notifications
You must be signed in to change notification settings - Fork 8
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
Move NOMAD examples to plugin #448
Conversation
Such a solution would from my point of view be the most logical place to locate these examples, i.e. together with each reader. In any case, we should follow a SSOT approach for these examples. |
Absolutely, because as of now, we not only need to update them in NORTH and in nomad-FAIR, but also in all distribution branches in |
It feels a bit strange having them in NORTH at all, because there is not a 1:1 mapping of NORTH tools and pynxtools plugins, i.e. there is not a tool for every example/plugin, no? Maybe one could create another example repo with sub-repos for each example, which then gets submoduled where needed? However, then one still needs to update the submodule commits all the time, so also not ideal. |
I think what we could do is have the pynxtools plugins be the SSOT place that have these folders with the example files that shall be used in NOMAD. Then instead of having the files also in NORTH, we can just get them from GitHub in the NORTH CI/CD for testing. And for the NORTH tools that don't have a directly correspondant plugin, we just don't use this. Then for the NOMAD examples, we use the |
I still think we need to define the
|
I think we can just use https://download-directory.github.io to directly get the examples from a different repo as a zip file. Or we use |
This sounds like a reasonable approach. With the requirements for the entry points, I am not sure I understand this well enough |
What I hadn't figured out: we can just have the entry points for the examples in the individual plugins, see FAIRmat-NFDI/pynxtools-mpes#32. That makes it so much cleaner because the data can just live in the plugin repo and we don't need to use the |
.github/workflows/plugin_test.yaml
Outdated
@@ -22,31 +22,31 @@ jobs: | |||
include: | |||
|
|||
- plugin: pynxtools-ellips | |||
branch: main | |||
branch: bring-in-examples |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reminder to revert
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is what I noticed, you may want to confirm that.
42a7bad
to
c8c6f75
Compare
1efe5c6
to
c8177f5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Co-authored-by: RubelMozumder <[email protected]>
Co-authored-by: RubelMozumder <[email protected]>
2c3dbc1
to
3959896
Compare
EDIT: we can just have the data and the nomad plugin entrypoint within the plugins themselves
ExampleUploadEntryPoint