@kwryankrattiger looks like a good refactor, few things:

• What does DAV stands for?
• How about making a more verbose PR description
• It seems to be rebased on the top of other contributions, are those from the PR referenced in the description of this issue?

@vicentebolea

* What does DAV stands for?

Data and Visualization, @chuckatkins recommended the acronym for use in the SDK package.

* It seems to be rebased on the top of other contributions, are those from the PR referenced in the description of this issue?

There is one by me that is the main refactor the rest of them are from a rebase onto spack/develop, I think I may have rebased things in the wrong order, I will re-propagate all of the rebases for this branch.

+1 after fixing style check

@spackbot run pipeline

I had a problem triggering the pipeline.

@spackbot run pipeline

I had a problem triggering the pipeline.

The hpx package failed on gitlab with:

/builds/spack/spack/var/spack/repos/builtin/packages/hpx/package.py:166, in instrumentation_args:
        165    def instrumentation_args(self):
  >>    166        for value in self.variants['instrumentation'].values:
        167            if value == 'none':
        168                continue
        169

I think that was fixed in 1dcabdbc8d, in which case this PR needs a rebase.

@adamjstewart Did you intend to block this merge? This is ready now after rebasing on #27784 merged which fixed enabling ADIOS2 with ParaView.

Hmm, I don't remember dismissing @chuckatkins review. Usually when I do this it's because someone requested changes and the changes were added but the person never re-reviewed. Reviews also get dismissed when you rebase, maybe that's what happened? Anyway, no problems with this being merged from my side.