Skip to content

visit: add v3.5.0 - #6161

Open
vicentebolea wants to merge 3 commits into
spack:developfrom
vicentebolea:update-visit
Open

visit: add v3.5.0#6161
vicentebolea wants to merge 3 commits into
spack:developfrom
vicentebolea:update-visit

Conversation

@vicentebolea

Copy link
Copy Markdown
Member

No description provided.

@spackbot-triage spackbot-triage Bot added new-version Modifications to packages' `depends_on()` directives update-package Modifications to packages in the repository labels Aug 20, 2026
@spackbot-triage
spackbot-triage Bot requested a review from cyrush August 20, 2026 01:30
@vicentebolea
vicentebolea force-pushed the update-visit branch 2 times, most recently from 41d7c4f to 8fc71a6 Compare August 22, 2026 02:43
@spackbot-triage spackbot-triage Bot added the dependencies Modifications with a `depends_on()` directive label Aug 22, 2026
@cyrush

cyrush commented Aug 22, 2026

Copy link
Copy Markdown
Member

One challenge is that VisIt 3.5.0 uses Qt6, and the Qt spack packages were refactored into smaller packages starting with Qt6 -- due to the modular changes between Qt5 and Qt6

@vicentebolea
vicentebolea force-pushed the update-visit branch 3 times, most recently from 72e122b to da16186 Compare August 22, 2026 21:31
@vicentebolea

Copy link
Copy Markdown
Member Author

@cyrush CI passed! 🎉

@cyrush

cyrush commented Aug 24, 2026

Copy link
Copy Markdown
Member

Oh wow, how do we see the results of those visit builds?
(I couldn't find them in gitlab ci easily)

@vicentebolea

Copy link
Copy Markdown
Member Author

Oh wow, how do we see the results of those visit builds? (I couldn't find them in gitlab ci easily)

Good catch, the changes in the deps made the concretizer to prefer visit 3.4.1 which in turn resulted in no new builds. I am temporarily forcing visit 3.5.0 in the dav-sdk stack to test this new version.

@spackbot-triage spackbot-triage Bot added the patch Modifications to packages' sets of defined patches label Aug 25, 2026
@RikkiButler20 RikkiButler20 changed the title visit: add latest version 3.5.0 visit: add v3.5.0 Aug 25, 2026
@kwryankrattiger

Copy link
Copy Markdown
Contributor

Looks like HDF5 API compatibility issue.

src/databases/OpenPMD/OpenPMDClasses/PMDFile.C:303:34: error: too many arguments to function 'herr_t H5Oget_info_by_name1(hid_t, const char*, H5O_info1_t*, hid_t)'

VisIt is setting the API version explicitly here, duplicate of the command line we use but that should be ok.

src/databases/OpenPMD/OpenPMDClasses/PMDParticle.h:26: warning: "H5_USE_18_API" redefined

@kwryankrattiger

Copy link
Copy Markdown
Contributor

I am temporarily forcing visit 3.5.0 in the dav-sdk stack to test this new version.

We should probably do that anyway, older versions of VTK are going to get removed before long and that means most of the versions of VisIt will also have to go. Forcing the latest is not a bad idea, at very least it will keep testing the VTK upper bound for VisIt in CI.

@vicentebolea

Copy link
Copy Markdown
Member Author

Looks like HDF5 API compatibility issue.

src/databases/OpenPMD/OpenPMDClasses/PMDFile.C:303:34: error: too many arguments to function 'herr_t H5Oget_info_by_name1(hid_t, const char*, H5O_info1_t*, hid_t)'

VisIt is setting the API version explicitly here, duplicate of the command line we use but that should be ok.

src/databases/OpenPMD/OpenPMDClasses/PMDParticle.h:26: warning: "H5_USE_18_API" redefined

Thanks for pointing this out, how did you get this errors? in the CI job it says: Job's log exceeded limit of

@vicentebolea
vicentebolea force-pushed the update-visit branch 2 times, most recently from 912b41a to b27f255 Compare August 27, 2026 01:15
@kwryankrattiger

kwryankrattiger commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Thanks for pointing this out, how did you get this errors? in the CI job it says: Job's log exceeded limit of

You can get the full logs from the artifacts under jobs_scratch_dir/<stack-name>/logs/spack-build-out.txt.gz. If the job fails for infrastructure reasons that isn't always available, but when it is a build error it should be there.

Here is the full log of the latest failure.
spack-build-out.visit.txt.gz

src/databases/OpenPMD/OpenPMDClasses/PMDIteration.C:143:50: error: too many arguments to function 'herr_t H5Oget_info_by_name1(hid_t, const char*, H5O_info1_t*, hid_t)'
...
src/databases/OpenPMD/OpenPMDClasses/PMDIteration.C:363:58: error: too many arguments to function 'herr_t H5Oget_info_by_name1(hid_t, const char*, H5O_info1_t*, hid_t)'

This looks like the signature for H5Oget_info_by_name2 which was introduced in 1.10.3, so I am curious how this was building before.

VVolumePlot is only created when the viewer/GUI components are
built, but plots/Volume/CMakeLists.txt unconditionally references it
in vtk_module_autoinit, breaking ~gui builds of visit@3.5:.
@kwryankrattiger
kwryankrattiger enabled auto-merge (squash) August 28, 2026 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Modifications with a `depends_on()` directive new-version Modifications to packages' `depends_on()` directives patch Modifications to packages' sets of defined patches update-package Modifications to packages in the repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants