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.

Sign up for GitHub

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

Jump to bottom

Patch for xrootd with derived variables #4200

Merged
merged 1 commit into from
Jun 11, 2024
Merged

Patch for xrootd with derived variables #4200

merged 1 commit into from
Jun 11, 2024

Conversation

anagainaru
Copy link
Contributor

The error we are getting when both XRootD and the derived variables are both ON:

Undefined symbols for architecture arm64:
  "adios2::derived::Curl3DFunc(std::__1::vector<adios2::derived::DerivedData, std::__1::allocator<adios2::derived::DerivedData>>, adios2::DataType)", referenced from:
      __GLOBAL__sub_I_XrdSsiSvService.cpp in XrdSsiSvService.cpp.o
  "adios2::derived::CurlDimsFunc(std::__1::vector<std::__1::vector<unsigned long, std::__1::allocator<unsigned long>>, std::__1::allocator<std::__1::vector<unsigned long, std::__1::allocator<unsigned long>>>>)", referenced from:
      __GLOBAL__sub_I_XrdSsiSvService.cpp in XrdSsiSvService.cpp.o
  "adios2::derived::SameDimsFunc(std::__1::vector<std::__1::vector<unsigned long, std::__1::allocator<unsigned long>>, std::__1::allocator<std::__1::vector<unsigned long, std::__1::allocator<unsigned long>>>>)", referenced from:
      __GLOBAL__sub_I_XrdSsiSvService.cpp in XrdSsiSvService.cpp.o
  "adios2::derived::MagnitudeFunc(std::__1::vector<adios2::derived::DerivedData, std::__1::allocator<adios2::derived::DerivedData>>, adios2::DataType)", referenced from:
      __GLOBAL__sub_I_XrdSsiSvService.cpp in XrdSsiSvService.cpp.o
  "adios2::derived::AddFunc(std::__1::vector<adios2::derived::DerivedData, std::__1::allocator<adios2::derived::DerivedData>>, adios2::DataType)", referenced from:
      __GLOBAL__sub_I_XrdSsiSvService.cpp in XrdSsiSvService.cpp.o
ld: symbol(s) not found for architecture arm64

@anagainaru @eisenhauer
Patch for when both xrootd and derived variables are activated
caed958 
Co-authored-by: Greg Eisenhauer <eisen@cc.gatech.edu>
eisenhauer
eisenhauer approved these changes Jun 11, 2024
View reviewed changes
Copy link
Member

@eisenhauer eisenhauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to fix the problem where I saw it evidenced (OSX with both derived and xrood enabled).

@eisenhauer eisenhauer merged commit 8cd2d94 into ornladios:master Jun 11, 2024
39 checks passed
@anagainaru anagainaru deleted the xrootd-derived branch June 11, 2024 12:54
guj pushed a commit to guj/ADIOS2 that referenced this pull request Jun 18, 2024
@anagainaru @eisenhauer @guj
Patch for when both xrootd and derived variables are activated (ornla…
8371475 
…dios#4200)

Co-authored-by: Greg Eisenhauer <eisen@cc.gatech.edu>
@anagainaru anagainaru mentioned this pull request Jun 18, 2024
Merged
@anagainaru anagainaru added this to the v2.10.2 milestone Oct 1, 2024
vicentebolea pushed a commit to vicentebolea/ADIOS2 that referenced this pull request Oct 23, 2024
@anagainaru @vicentebolea
Patch for when both xrootd and derived variables are activated (ornla…
5ca0774 
…dios#4200)

Co-authored-by: Greg Eisenhauer <eisen@cc.gatech.edu>
(cherry picked from commit 8cd2d94)
@vicentebolea vicentebolea mentioned this pull request Oct 23, 2024
Merged
vicentebolea pushed a commit to vicentebolea/ADIOS2 that referenced this pull request Oct 23, 2024
@anagainaru @vicentebolea
Patch for when both xrootd and derived variables are activated (ornla…
5f84b69 
…dios#4200)

Co-authored-by: Greg Eisenhauer <eisen@cc.gatech.edu>
(cherry picked from commit 8cd2d94)
vicentebolea pushed a commit to vicentebolea/ADIOS2 that referenced this pull request Oct 23, 2024
@anagainaru @vicentebolea
Patch for when both xrootd and derived variables are activated (ornla…
3167e39 
…dios#4200)

Co-authored-by: Greg Eisenhauer <eisen@cc.gatech.edu>
(cherry picked from commit 8cd2d94)
vicentebolea pushed a commit to vicentebolea/ADIOS2 that referenced this pull request Oct 24, 2024
@anagainaru @vicentebolea
Patch for when both xrootd and derived variables are activated (ornla…
e004f9f 
…dios#4200)

Co-authored-by: Greg Eisenhauer <eisen@cc.gatech.edu>
(cherry picked from commit 8cd2d94)
vicentebolea pushed a commit to vicentebolea/ADIOS2 that referenced this pull request Oct 24, 2024
@anagainaru @vicentebolea
Patch for when both xrootd and derived variables are activated (ornla…
c347505 
…dios#4200)

Co-authored-by: Greg Eisenhauer <eisen@cc.gatech.edu>
(cherry picked from commit 8cd2d94)
vicentebolea pushed a commit to vicentebolea/ADIOS2 that referenced this pull request Oct 24, 2024
@anagainaru @vicentebolea
Patch for when both xrootd and derived variables are activated (ornla…
e53bf99 
…dios#4200)

Co-authored-by: Greg Eisenhauer <eisen@cc.gatech.edu>
(cherry picked from commit 8cd2d94)
@vicentebolea vicentebolea mentioned this pull request Oct 26, 2024
Merged
vicentebolea added a commit that referenced this pull request Oct 26, 2024
@vicentebolea
Merge branch 'release_210'
450962f 
* 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
Reviewers

@eisenhauer eisenhauer eisenhauer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.10.2
Development

Successfully merging this pull request may close these issues.
3 participants
@anagainaru @eisenhauer @vicentebolea