Open thewtex opened 1 year ago
I am not sure why we are seeing these warnings:
https://open.cdash.org/viewBuildError.php?type=1&buildid=8385181
:confused:
Was the word "intentially" intentionally used in the commit message?
Was the word "intentially" intentionally used in the commit message?
No :smile: :laughing:
I am not sure why we are seeing these warnings: https://open.cdash.org/viewBuildError.php?type=1&buildid=8385181
I do not know either Matt. Looks like the latest build points at some additional warnings: https://open.cdash.org/viewBuildError.php?type=1&buildid=8386005 https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/actions/runs/3870763866/jobs/6597968633#step:9:12602
It looks as if finding the same breathlink
s would bother Sphinx:
e.g.
https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/blame/468ee04f79c5386393db1aa6929200f2eeb6126c/src/Core/Common/CreateVectorImage/Documentation.rst#L34
and
https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/blame/468ee04f79c5386393db1aa6929200f2eeb6126c/src/Core/Common/CastVectorImageToAnotherType/Documentation.rst#L34
It looks like the class documentation block is not correctly processed by Doxygen for some classes e.g. https://github.com/InsightSoftwareConsortium/ITK/blob/87a66198e11920eeb04db1d6125a37e83cf4ebf6/Modules/Numerics/Statistics/include/itkSampleToHistogramFilter.h
Cannot be found under the ITK namespace in: https://itk.org/Doxygen53/html/annotated.html or under the Statistics
namespace: https://itk.org/Doxygen53/html/namespaceitk_1_1Statistics.html
Previously noticed these tag file warnings/errors last year in https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/issues/362, the workaround at that time was to pin to v5.2.0
tag files which we obviously no longer want to do. I'm not sure what causes these issues. It sounds like the warnings may have stemmed from a doxygen version bump, the dates given in that issue discussion may aid in investigation.
Interestingly one of the classes at issue, SampleToHistogramFilter
, was present in the ITK 5.1/5.2 Doxygen:
https://itk.org/Doxygen51/html/classitk_1_1Statistics_1_1SampleToHistogramFilter.html
https://itk.org/Doxygen52/html/classitk_1_1Statistics_1_1SampleToHistogramFilter.html
https://itk.org/Doxygen52/html/namespaceitk_1_1Statistics.html https://itk.org/Doxygen51/html/namespaceitk_1_1Statistics.html vs https://itk.org/Doxygen53/html/namespaceitk_1_1Statistics.html
A blame shows 5 commits since ITK 5.1. The one that I'd say may be related is the last one, which I contributed myself: https://github.com/InsightSoftwareConsortium/ITK/commits/87a66198e11920eeb04db1d6125a37e83cf4ebf6/Modules/Numerics/Statistics/include/itkSampleToHistogramFilter.h
Other classes that went through the same change are present in Doxygen, though: https://itk.org/Doxygen53/html/classitk_1_1VTKImageExport.html
A couple of 2 other points: https://raw.githubusercontent.com/InsightSoftwareConsortium/ITKSphinxExamples/5018e60c875c44ca6bdf6198b798b0010ef8e3f7/src/Numerics/Statistics/CreateHistogramFromListOfMeasurements/Documentation.rst https://raw.githubusercontent.com/InsightSoftwareConsortium/ITKSphinxExamples/5018e60c875c44ca6bdf6198b798b0010ef8e3f7/src/Core/Common/CastVectorImageToAnotherType/Documentation.rst
histogram
class in the index that is surely not found (will submit a patch set)pair: cast; VectorImage
in the index that I am not sure was meant that way (the code shows the use of itk::CastImageFilter) (will submit a patch set)Show a histogram class in the index that is surely not found (will submit a patch set) Show a pair: cast; VectorImage in the index that I am not sure was meant that way (the code shows the use of itk::CastImageFilter) (will submit a patch set)
PR #412. One thing that I wonder is why the second one has a single
then a pair
entry with the latter containing an element that is already in single
. I did not find the behavior documented in Sphinx. @thewtex @tbirdso ?
PR https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/pull/412. One thing that I wonder is why the second one has a single then a pair entry with the latter containing an element that is already in single. I did not find the behavior documented in Sphinx. @thewtex @tbirdso ?
Nevermind. I figured out what the purpose and effect is.
PR #412. One thing that I wonder is why the second one has a single then a pair entry with the latter containing an element that is already in single. I did not find the behavior documented in Sphinx. @thewtex @tbirdso ?
Nevermind. I figured out what the purpose and effect is.
Hi @jhlegarreta , I am not clear on what these tags do, was there documentation that helped you or could you briefly summarize what you found?
Hi @jhlegarreta , I am not clear on what these tags do, was there documentation that helped you or could you briefly summarize what you found?
They serve to create the index: https://examples.itk.org/genindex.html
For the case at issue, it single
creates the VectorImage
entry we see in V
(hover on the class name); the pair
creates the entry as a composite entry by subindexing both as cast/VectorImage and VectorImage/cast.
It seems we should rebase after @jhlegarreta 's related patches are in, check to see if the warnings actually cause issues in the generated documentation, ignore them if not, and keep https://github.com/InsightSoftwareConsortium/ITKSphinxExamples/issues/362 open to track the issue.
Rebasing on master
It's a CMake warning that is making the ubuntu-22.04 documentation check fail: https://open.cdash.org/build/8867148/configure
I thought we had fixed this one.
And some other Doxygen warnings: https://open.cdash.org/viewBuildError.php?type=1&buildid=8867148
It's a CMake warning that is making the ubuntu-22.04 documentation check fail: https://open.cdash.org/build/8867148/configure
I thought we had fixed this one.
Does not look like we fixed properly in other places either: InsightSoftwareConsortium/ITK#3846
And some other Doxygen warnings: https://open.cdash.org/viewBuildError.php?type=1&buildid=8867148
Not sure what the problem is with itk::AddImageFilter
and itk::SampleToHistogramFilter
:
itk
namespace.Maybe the warnings in the GHA builds https://github.com/InsightSoftwareConsortium/ITKDoxygen/actions/runs/5720889621/job/15501589831 and/or in the dashboard https://open.cdash.org/viewBuildError.php?type=1&buildid=8879198
need attention. A quick look did not result in meaningful insight about the errors.
Some further comments:
AddImageFilter
as the name of a group, enum or namespace in here: https://itk.org/Doxygen/html/annotated.html. Maybe Doxygen is trying to create a group for the Ternary, Nary, Weighted variants?AddImageFilter
; but not for the Ternary, Nary, Weighted variants.PowImagFilter
has the same issues.I've already run out of ideas Matt.
@jhlegarreta thanks for diving in here!
The inheritance and collaboration diagram generation is also failing for AddImageFilter; but not for the Ternary, Nary, Weighted variants. PowImagFilter has the same issues.
Based on this evaluation / observation, I noticed an difference in AddImageFilter, PowImageFilter vs the Nary, Ternary, Weighted AddImageFilters that is removed here:
https://github.com/InsightSoftwareConsortium/ITK/pull/4208
Crossing fingers that addresses these issues :crossed_fingers: .
Note: This is intentionally not updated to the ITK source code version so we can bump the source version separately.