HealthBioscienceIDEAS / Medical-Image-Registration-Short-Course

⚕️Short Course on Medical Image Registration
https://healthbioscienceideas.github.io/Medical-Image-Registration-Short-Course/
Other
3 stars 0 forks source link

1 switch to carpentriessandpaper #22

Closed davecash75 closed 2 months ago

davecash75 commented 2 months ago

Description

Fixes #1: Now that sandpaper change has been implemented, I have adjusted not only the config.yaml file as @milanmlft suggested, but also the setting up instructions, which included a lot of how the repository was originally created. Since that has already been done, new users won't have to do that anymore.

Type of change

Suggested Checklist

github-actions[bot] commented 2 months ago

Thank you!

Thank you for your pull request :smiley:

:robot: This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

Rendered Changes

:mag: Inspect the changes: https://github.com/HealthBioscienceIDEAS/Medical-Image-Registration-Short-Course/compare/md-outputs..md-outputs-PR-22

The following changes were observed in the rendered markdown documents:

 md5sum.txt | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)
What does this mean? If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible. This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

:stopwatch: Updated at 2024-09-04 13:25:10 +0000

davecash75 commented 2 months ago

Hi @milanmlft I think this is ready for a quick re-review. I tested it with @jamie-mcclelland on his Windows and some final changes were made.

milanmlft commented 2 months ago

Oh quick afterthought, it might be a good idea to clear the GHA caches, because I can see it still has the IDEAS-sandpaper fork installed. Clearing the cache should force re-installation.

davecash75 commented 2 months ago

Thanks @milanmlft - I've deleted the caches as suggested.