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

JOSS review item: Examples #72

Open
Matthew-Jennings opened this issue Apr 4, 2024 · 3 comments · Fixed by #86
Open

JOSS review item: Examples #72

Matthew-Jennings opened this issue Apr 4, 2024 · 3 comments · Fixed by #86

Comments

@Matthew-Jennings
Copy link
Contributor

Hi @alexzwanenburg. Great stuff putting this project together! Apologies for the delay in review, I hope you find the feedback useful. I'll separate my review items into individual issues. If you address an issue in a PR, please link the issue within the PR or vice versa.

The examples provided - forgive me - add almost nothing to the API documentation. I really think it is worth fleshing these out, and probably much more so than, say, separating out the installation instructions into a separate page from the README. In my experience, users heavily rely on examples to grok a library.

Well written examples, IMHO, should depict near end-to-end usage of MIRP's functionality, using actual input data (mock is okay, even preferred, but a placeholder "path/to/input/file" is usually not) and frequently, explicitly printing or plotting outputs along the way. To get an idea, have a look at this not-even-that-great-example in another repo I help maintain, PyMedPhys.

Presumably the meat of most of your conceivable examples is already present in your test suite?

Jupyter notebooks are excellent for this. If you struggle to integrate Jupyter notebooks into the docs, simply include an example folder of them and link to them from the docs.

Let me know what you think, happy to discuss.

alexzwanenburg added a commit that referenced this issue Apr 5, 2024
Aim is to transition quick_start.rst into the notebook and create a working example.
@alexzwanenburg
Copy link
Member

Thanks for the pointers! I have added a first tutorial to documentation. Check it out here:
https://oncoray.github.io/mirp/tutorial_compute_radiomics_features_mr.html

I am leaving this issue open as more tutorials can and should be added.

@alexzwanenburg alexzwanenburg linked a pull request May 14, 2024 that will close this issue
@alexzwanenburg
Copy link
Member

Reopened for JOSS review.

@alexzwanenburg
Copy link
Member

A second tutorial may be found here: https://oncoray.github.io/mirp/tutorial_apply_image_filter.html

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 a pull request may close this issue.

2 participants