Skip to content

Conversation

@ktf
Copy link
Member

@ktf ktf commented Jun 17, 2025

No description provided.

@github-actions
Copy link
Contributor

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@ktf ktf changed the title DPL: move to std::pmr where possible Move to std::pmr where possible Jun 17, 2025
@alibuild
Copy link
Collaborator

Error while checking build/O2/fullCI_slc9 for 0cde38f at 2025-06-18 00:40:

## sw/BUILD/O2-latest/log
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:111:27: error: 'polymorphic_allocator' was not declared in this scope
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:111:57: error: template argument 2 is invalid
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:111:92: error: expected primary-expression before '>' token
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:111:93: error: expected primary-expression before '{' token
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:112:7: error: request for member 'emplace_back' in 'v', which is of non-class type 'int'
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:113:7: error: request for member 'emplace_back' in 'v', which is of non-class type 'int'
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:114:7: error: request for member 'emplace_back' in 'v', which is of non-class type 'int'
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:115:30: error: invalid types 'int[int]' for array subscript
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:126:27: error: 'polymorphic_allocator' was not declared in this scope
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:126:57: error: template argument 2 is invalid
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:126:92: error: expected primary-expression before '>' token
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:126:93: error: expected primary-expression before '{' token
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:127:7: error: request for member 'emplace_back' in 'v', which is of non-class type 'int'
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:128:7: error: request for member 'emplace_back' in 'v', which is of non-class type 'int'
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:129:7: error: request for member 'emplace_back' in 'v', which is of non-class type 'int'
/sw/SOURCES/O2/14417-slc9_x86-64/0/DataFormats/MemoryResources/test/testMemoryResources.cxx:130:30: error: invalid types 'int[int]' for array subscript
/sw/slc9_x86-64/FairMQ/v1.10.0-local1/include/fairmq/MemoryResourceTools.h:35:28: error: request for member 'get_allocator' in 'container', which is of non-class type 'int'
/sw/slc9_x86-64/FairMQ/v1.10.0-local1/include/fairmq/MemoryResourceTools.h:41:43: error: request for member 'size' in 'container', which is of non-class type 'int'
/sw/slc9_x86-64/FairMQ/v1.10.0-local1/include/fairmq/MemoryResourceTools.h:41:52: error: 'int' is not a class, struct, or union type
/sw/slc9_x86-64/FairMQ/v1.10.0-local1/include/fairmq/MemoryResourceTools.h:45:13: error: 'int' is not a class, struct, or union type
/sw/slc9_x86-64/FairMQ/v1.10.0-local1/include/fairmq/MemoryResourceTools.h:46:27: error: request for member 'data' in 'container', which is of non-class type 'int'
/sw/slc9_x86-64/FairMQ/v1.10.0-local1/include/fairmq/MemoryResourceTools.h:60:19: error: request for member 'data' in 'container', which is of non-class type 'int'
ninja: build stopped: subcommand failed.

Full log here.

@ktf ktf changed the title Move to std::pmr where possible DPL: move to std::pmr where possible Jun 18, 2025
@ktf
Copy link
Member Author

ktf commented Jun 18, 2025

@Barthelemy @vascobarroso This will require a new version of FairMQ (already merged in alidist). Is it ok to merge or do you plan to do a release soon with new O2 and old FairMQ?

@vascobarroso
Copy link
Member

@Barthelemy @vascobarroso This will require a new version of FairMQ (already merged in alidist). Is it ok to merge or do you plan to do a release soon with new O2 and old FairMQ?

I think it is ok to merge, we don't plan any new releases in the next 2 weeks until the Oxygen run is finished.
But there is an issue with StfBuilder and the new FairMQ, Lubos is looking into it but until we have a fix we are stuck.

@alibuild
Copy link
Collaborator

alibuild commented Jul 5, 2025

Error while checking build/O2/fullCI_slc9 for d0b7090 at 2025-07-15 06:15:

No log files found

Full log here.

@ktf
Copy link
Member Author

ktf commented Jul 24, 2025

The issue with DataDistribution has been understood and fixed. See 14537.

@ktf
Copy link
Member Author

ktf commented Aug 6, 2025

Since everything seems to be working and the DataDistribution got tested by @lkrcal on staging, I am merging this given it's basically just cosmetics (i.e. FairMQ in alidist is already using std::pmr).

@ktf ktf merged commit 83467b3 into AliceO2Group:dev Aug 6, 2025
17 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants