Jon Haitz Legarreta Gorroño

Add a pre-commit hook to format the Markdown files

1

Adding a pre-commit hook that is able to ensure that the Markdown files have a consistent format and follow the carpentries style would be a great addition and would allow...

Add exercise statements and solutions notebooks

2

The introduction and DTI episodes have each an exercise so that learners can demonstrate the acquired skills; however, the exercises are only written in the Markdown files, and they are...

At times, we manually edit the raw Jupyter notebooks (like https://github.com/carpentries-incubator/SDC-BIDS-dMRI/pull/137#discussion_r613573684) and break the notebooks and they effectively become unusable. Although the CI tests in place would potentially detect such...

Use the `scene.reset_camera()` call in the `fury` camera setting methods when the issue no. 38 in `fury` gets fixed. Cross-referencing https://github.com/carpentries-incubator/SDC-BIDS-dMRI/pull/121#discussion_r606852029.

Improve tests by using the ITK testing macros.

2

The existing tests should be improved: - The ITK testing macros should be used (e.g. `TRY_EXPECT_NO_EXCEPTION`, `EXERCISE_BASIC_OBJECT_METHODS`, etc.) to avoid boilerplate code.

COMP: Fix implicit copy constructor definition deprecation warnings

3

Fix implicit copy constructor definition deprecation warnings in image/mesh file reader exception classes by using explicitly the compiler-provided default implementation (together with the assignment assignment and move semantics). Fixes: ```...

The existing tests should be improved: - ITK testing conventions on the naming of the test files should be applied. - The ITK testing macros should be used (e.g. `TRY_EXPECT_NO_EXCEPTION`,...

Defaulting copy constructor, copy assignment, move constructor, and move assignment functions

8

### Description The dashboard site `RogueResearch22` is consistently raising warnings about classes declaring implicit copy constructors, copy assignment operators, etc. https://learn.microsoft.com/bs-latn-ba/cpp/error-messages/compiler-warnings/c5267?view=msvc-150#remarks Over the last few days some PRs have addressed...

For some reason, the `Appendices` chapter bookmarks are not generated in the PDF file, although the sections are available in the table of contents. For an easier navigation, it would...

Add a Python style guide. The [System Overview Wrapping](https://github.com/InsightSoftwareConsortium/ITKSoftwareGuide/blob/bf60afa4274dc5c285ac5c8b173a440866def9c8/SoftwareGuide/Latex/Architecture/SystemOverview.tex#L755) section contains [some example code](https://github.com/InsightSoftwareConsortium/ITKSoftwareGuide/blob/bf60afa4274dc5c285ac5c8b173a440866def9c8/SoftwareGuide/Latex/Architecture/SystemOverview.tex#L792) of what an ITK Python code looks like, but we are lacking some guidelines on how...