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

Backport #3759 to Release 2.10 branch. #4341

Merged

Conversation

pnorbert
Copy link
Contributor

(cherry-picked from master branch #4340)

Organize the processes into groups so that the two steps of metadata aggregation has more or less the same number or participants. This replaces in-node aggregation in first step. The new strategy balances the size of metadata gathered in the two steps.

…aster branch ornladios#4340)

Organize the processes into groups so that the two steps of metadata aggregation has more or less the same number or participants. This replaces in-node aggregation in first step. The new strategy balances the size of metadata gathered in the two steps.
@pnorbert
Copy link
Contributor Author

@eisenhauer @vicentebolea Any idea why the macos configuration fails for the release_210 branch? This is the second run. Complaining about gnu 11 when using clang.

@eisenhauer
Copy link
Member

If I were guessing, I'd say that the runner images have changed, master CI has been updated to fix it and release_210 hasn't. Would need to look for the PR that fixed it, probably merged to master but not backported.

@eisenhauer
Copy link
Member

I haven't sorted out the PR that changed it yet, but we no longer build on macos-11 or -12 in master. I'm going to bypass branch protections to go ahead and merge this PR, but if we're going to keep updating release_210, we need to sort the CI.

@eisenhauer eisenhauer merged commit 5283b27 into ornladios:release_210 Sep 11, 2024
36 of 39 checks passed
@pnorbert pnorbert deleted the backport_PR3750_to_release_210 branch September 16, 2024 17:36
@vicentebolea vicentebolea mentioned this pull request Sep 18, 2024
vicentebolea added a commit to vicentebolea/ADIOS2 that referenced this pull request Oct 24, 2024
This was referenced Oct 24, 2024
vicentebolea added a commit that referenced this pull request Oct 26, 2024
* release_210: (33 commits)
  Bump version to v2.10.2
  Merge pull request #4379 from vicentebolea/increace-cmake-version-dep-blosc
  Fix missing int64_t definition (#4365)
  Merge pull request #4346 from anagainaru/memSelLayout
  Merge pull request #4343 from anagainaru/gpuSpan
  Compress the lossless part of MGARD data with Zstd. It was uncompressed until now. (#4339)
  Fix an issue when reading blocks in streaming mode, when some producers did not write any block to output. (#4332)
  Fix in python API:  (#4331)
  Merge pull request #4303 from anagainaru/build-frontier
  Merge pull request #4293 from anagainaru/check_parser_lexer
  Derived variables of type StoreData should not record the expression string (#4247)
  Update ReadMe.md (#4243)
  Switch from ROCM_VERSION_MAJOR to HIP_VERSION_MAJOR (#4222)
  Merge pull request #4207 from anagainaru/hide-symbols
  Patch for when both xrootd and derived variables are activated (#4200)
  Add defines for BP3, BP4 and BP5 (#4191)
  Overload NdCopy instead of adding a new argument for the 2.10 release
  Revert "Backport #3759 to Release 2.10 branch. (cherry-picked from master branch #4340) (#4341)"
  Revert "Merge pull request #4189 from pnorbert/campaign-s3-support"
  ci,python: enable python win2022 serial build
  ...
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.

2 participants