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

Media wiki migration second set #109

Merged
merged 10 commits into from
Jun 10, 2019
Merged

Conversation

hjmjohnson
Copy link
Member

Most of the media wiki examples are migrated and building with the sphinx examples hierarchy now.

SphinxExamples requires VTK with compatible python
bindings for python 3.6 or greater.
When building ITKEx, ensure that the appropriate level of
compiler standard is used.
@hjmjohnson hjmjohnson added the type:Enhancement Improvement of existing methods or implementation label Jun 10, 2019
@hjmjohnson hjmjohnson self-assigned this Jun 10, 2019
Copy link
Member Author

@hjmjohnson hjmjohnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@thewtex @dzenanz This was a huge amount of work. We know it has more work to do but would like to get this integrated as a starting point for the next phase of updates.

Many of the new examples have tests that are failing. We are trying to figure out how to efficiently create these into passing tests.

Copy link
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! Also so big it is not feasible to review it in detail. I did notice a few things by glancing through it.

src/Core/Common/MultThreadOilPainting/CMakeLists.txt Outdated Show resolved Hide resolved
src/Core/Common/MultThreadOilPainting/Code.cxx Outdated Show resolved Hide resolved
@dzenanz
Copy link
Member

dzenanz commented Jun 10, 2019

Does this close #15?

@hjmjohnson hjmjohnson force-pushed the media-wiki-migration-second-set branch 2 times, most recently from fe569df to 347d91e Compare June 10, 2019 15:50
@hjmjohnson
Copy link
Member Author

@dzenanz I hit your quick review comments. I'm sure that there are many other items like those you identified, but as you said, this is a HUGE undertaking.

I believe that the current code with problems is worth merging so that we can work on the refinements as the next steps. The integration with ITK proper is MUCH easier with some iterative markers in these examples.

Additionally, it is sometimes easier to identify problems from the generated sphinx example pages than it is from the main codebase.

Hans

Please rebase and merge if you are content with my argument. The student "mseng10" is in a holding pattern this week until we identify a new stable working point.

@dzenanz dzenanz merged commit 17d6c6f into master Jun 10, 2019
@dzenanz dzenanz deleted the media-wiki-migration-second-set branch June 10, 2019 15:57
@hjmjohnson
Copy link
Member Author

@dzenanz Awesome! Thank you.

@dzenanz
Copy link
Member

dzenanz commented Jun 10, 2019

As this is a huge effort, I want to make it easier to you and @mseng10. Hoping everything will end up tidy, not too far into the future 😄

@hjmjohnson
Copy link
Member Author

@mseng10 is working on adding reference images this week.

@thewtex
Copy link
Member

thewtex commented Jun 19, 2019

@hjmjohnson @mseng10 amazing work!!! 🥇 🌮 🔢

jhlegarreta added a commit to jhlegarreta/ITKSoftwareGuide that referenced this pull request May 12, 2020
Miscellaneus fixes in the `Introduction` and `Installation` chapters:
- Fix the development environment names.
- Remove compiler support dates prior to 2016 from the compiler support
  table.
- Remove the deprecated reference to the Wiki Examples: examples are
  entirely hosted in https://itk.org/ITKExamples/ since a series of PRs
  including
  InsightSoftwareConsortium/ITKSphinxExamples#108 and
  InsightSoftwareConsortium/ITKSphinxExamples#109.
- Fix the `Utilities` and `Wrapping` parent folders.
- Tke advantage of the commit to order alphabetically the ITK's main
  folders.

Take advantage of the commit for other minor style changes (e.g. period at
the end of sentences, proper LaTex quote syntax, etc.).

Take advantage of the commit to limit the line length to 79 characters in
the ITK's folder structure bullet points.
jhlegarreta added a commit to jhlegarreta/ITKSoftwareGuide that referenced this pull request May 12, 2020
Miscellaneus fixes in the `Introduction` and `Installation` chapters:
- Fix the development environment names.
- Remove compiler support dates prior to 2016 from the compiler support
  table.
- Remove the deprecated reference to the Wiki Examples: examples are
  entirely hosted in https://itk.org/ITKExamples/ since a series of PRs
  including
  InsightSoftwareConsortium/ITKSphinxExamples#108 and
  InsightSoftwareConsortium/ITKSphinxExamples#109.
- Fix the `Utilities` and `Wrapping` parent folders.
- Tke advantage of the commit to order alphabetically the ITK's main
  folders.

Take advantage of the commit for other minor style changes (e.g. period at
the end of sentences, proper LaTex quote syntax, etc.).

Take advantage of the commit to limit the line length to 79 characters in
the ITK's folder structure bullet points.
jhlegarreta added a commit to jhlegarreta/ITKSoftwareGuide that referenced this pull request May 12, 2020
Miscellaneus fixes in the `Introduction` and `Installation` chapters:
- Fix the development environment names.
- Remove compiler support dates prior to 2016 from the compiler support
  table.
- Remove the deprecated reference to the Wiki Examples: examples are
  entirely hosted in https://itk.org/ITKExamples/ since a series of PRs
  including
  InsightSoftwareConsortium/ITKSphinxExamples#108 and
  InsightSoftwareConsortium/ITKSphinxExamples#109.
- Fix the `Utilities` and `Wrapping` parent folders.
- Mention the raw binary files and the SHA512 hash files in the `Testing`
  folder paragraph.
- Tke advantage of the commit to order alphabetically the ITK's main
  folders.

Take advantage of the commit for other minor style changes (e.g. period at
the end of sentences, proper LaTex quote syntax, etc.).

Take advantage of the commit to limit the line length to 79 characters in
the ITK's folder structure bullet points.
jhlegarreta added a commit to jhlegarreta/ITKSoftwareGuide that referenced this pull request May 13, 2020
Miscellaneus fixes in the `Introduction` and `Installation` chapters:
- Fix the development environment names.
- Remove compiler support dates prior to 2016 from the compiler support
  table.
- Remove the deprecated reference to the Wiki Examples: examples are
  entirely hosted in https://itk.org/ITKExamples/ since a series of PRs
  including
  InsightSoftwareConsortium/ITKSphinxExamples#108 and
  InsightSoftwareConsortium/ITKSphinxExamples#109.
- Fix the `Utilities` and `Wrapping` parent folders.
- Mention the raw binary files and the SHA512 hash files in the `Testing`
  folder paragraph.
- Take advantage of the commit to order alphabetically the ITK's main
  folders.

Take advantage of the commit for other minor style changes (e.g. period at
the end of sentences, proper LaTex quote syntax, etc.).

Take advantage of the commit to limit the line length to 79 characters in
the ITK's folder structure bullet points.
thewtex pushed a commit to jhlegarreta/ITKSoftwareGuide that referenced this pull request May 14, 2020
Miscellaneus fixes in the `Introduction` and `Installation` chapters:
- Fix the development environment names.
- Remove compiler support dates prior to 2016 from the compiler support
  table.
- Remove the deprecated reference to the Wiki Examples: examples are
  entirely hosted in https://itk.org/ITKExamples/ since a series of PRs
  including
  InsightSoftwareConsortium/ITKSphinxExamples#108 and
  InsightSoftwareConsortium/ITKSphinxExamples#109.
- Fix the `Utilities` and `Wrapping` parent folders.
- Mention the raw binary files and the SHA512 hash files in the `Testing`
  folder paragraph.
- Take advantage of the commit to order alphabetically the ITK's main
  folders.

Take advantage of the commit for other minor style changes (e.g. period at
the end of sentences, proper LaTex quote syntax, etc.).

Take advantage of the commit to limit the line length to 79 characters in
the ITK's folder structure bullet points.
jhlegarreta added a commit to jhlegarreta/ITKSoftwareGuide that referenced this pull request May 14, 2020
Miscellaneus fixes in the `Introduction` and `Installation` chapters:
- Fix the development environment names.
- Rework the compiler list: update to currently supported versions.
- Remove the compiler support/deprecation cycle table due to difficulties
  in keeping it up to date.
- Remove the deprecated reference to the Wiki Examples: examples are
  entirely hosted in https://itk.org/ITKExamples/ since a series of PRs
  including
  InsightSoftwareConsortium/ITKSphinxExamples#108 and
  InsightSoftwareConsortium/ITKSphinxExamples#109.
- Update the `vxl` link.
- Fix the `Utilities` and `Wrapping` parent folders.
- Mention the raw binary files and the SHA512 hash files in the `Testing`
  folder paragraph.
- Take advantage of the commit to order alphabetically the ITK's main
  folders.

Take advantage of the commit for other minor style changes (e.g. period at
the end of sentences, proper LaTex quote syntax, etc.).

Take advantage of the commit to limit the line length to 79 characters in
the ITK's folder structure bullet points.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:Enhancement Improvement of existing methods or implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants