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

CLI: Export Pydra interfaces #2605

Open
Lestropie opened this issue Mar 21, 2023 · 5 comments
Open

CLI: Export Pydra interfaces #2605

Lestropie opened this issue Mar 21, 2023 · 5 comments
Labels

Comments

@Lestropie
Copy link
Member

https://github.com/nipype/pydra

Similar to existing __print_usage_*__ hidden command-line functions, it should be possible to auto-export---from both C++ and Python---the requisite interface information for Pydra.

Pinging @tclose.

@Lestropie
Copy link
Member Author

Code links:

@tclose
Copy link
Contributor

tclose commented Mar 22, 2023

Ok, cool. So we would only need to write something in the mold of std::string full_usage() to print out the Pydra interfaces?

Could then look into a GH Action to update the Pydra package with new Mrtrix releases

@Lestropie
Copy link
Member Author

Correct.

For eg. online documentation, a script generates that documentation, and that same script is re-run in CI to check for differences, so that software interface and online documentation are guaranteed to have correspondence. So that content is encapsulated within the repository itself, including the versions that correspond to prior tagged releases. For Pydra, is there a preference for having that interface information embedded within the relevant repository vs. stored somewhere externally?

@tclose
Copy link
Contributor

tclose commented Mar 22, 2023

For Pydra, is there a preference for having that interface information embedded within the relevant repository vs. stored somewhere externally?

The standard place to store the interfaces is in their own repo, e.g. https://github.com/nipype/pydra-mrtrix3, typically because they are managed by people external to the tool's dev team, but storing them in the main MRtrix3 repo would be ideal IMO.

@tclose
Copy link
Contributor

tclose commented Mar 22, 2023

Just checked with the Mattermost channel, and storing them in the mrtrix repo is preferred if possible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants