Skip to content

Try upgrading VTK from 9.1.0 to 9.5.0#2411

Merged
akenmorris merged 15 commits into
masterfrom
try_vtk_9.5
Sep 2, 2025
Merged

Try upgrading VTK from 9.1.0 to 9.5.0#2411
akenmorris merged 15 commits into
masterfrom
try_vtk_9.5

Conversation

@akenmorris

Copy link
Copy Markdown
Contributor

No description provided.

They made a number of changes that alter the output:

    Improve accuracy of vtkWindowedSincPolyDataFilter

    Using Hamming windowing function resulted in inaccurate scaling of the output
    (up to about 1% error, either increase or decrease of the scale).

    This commit implements all three window functions that are described in Taubin's paper and adds Nuttal window
    and changes the default to Nuttal to eliminate the scaling error.

    Note that when passband is set to very small value then shrinkage may occur regardless of the choice of window function.
https://github.com/AcademySoftwareFoundation/openvdb/pull/1977/files

I'm doing the patch instead of upgrading because upgrading requires
forcing a new version of GCC which will cascade other dependency
upgrades.  We may end up doing them, but we need to do this one step
at a time and I need the CI/CD working in order to test things.
/Users/runner/miniconda3/envs/shapeworks/include/boost/mpl/aux/integral_wrapper.hpp:73:31: error: integer value -1 is outside the valid range of values [0, 3] for the enumeration type 'int_float_mixtureenum' [-Wenum-constexpr-conversion]
2025-09-01T23:22:49.3891250Z /Users/runner/build/ITK/Modules/ThirdParty/PNG/src/itkpng/pngpriv.h:527:16: fatal error: 'fp.h' file not found
2025-09-01T23:22:49.3992130Z   527 | #      include <fp.h>
2025-09-01T23:22:49.4092720Z       |                ^~~~~~

It seems to be missing now with Clang17
…ror:

2025-09-02T09:23:00.8754420Z /Users/runner/miniconda3/envs/shapeworks/include/boost/mpl/aux_/integral_wrapper.hpp:73:31: error: integer value -1 is outside the valid range of values [0, 3] for the enumeration type 'int_float_mixture_enum' [-Wenum-constexpr-conversion]
2025-09-02T09:23:00.8755440Z /Users/runner/miniconda3/envs/shapeworks/include/boost/mpl/aux_/static_cast.hpp:24:47: note: expanded from macro 'BOOST_MPL_AUX_STATIC_CAST'
2025-09-02T09:23:00.8756050Z    24 | #   define BOOST_MPL_AUX_STATIC_CAST(T, expr) static_cast<T>(expr)
2025-09-02T09:23:00.8756400Z       |                                               ^
2025-09-02T09:23:00.8756680Z 2 errors generated.
@akenmorris akenmorris merged commit 5e33c32 into master Sep 2, 2025
2 of 5 checks passed
@akenmorris akenmorris deleted the try_vtk_9.5 branch September 2, 2025 21:05
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 this pull request may close these issues.

1 participant