Skip to content

Fixes in GRP DCS processor#12887

Merged
shahor02 merged 2 commits intoAliceO2Group:devfrom
shahor02:pr_dcsgrp
Mar 20, 2024
Merged

Fixes in GRP DCS processor#12887
shahor02 merged 2 commits intoAliceO2Group:devfrom
shahor02:pr_dcsgrp

Conversation

@shahor02
Copy link
Collaborator

Warn about empty collimator or env.vars object.
Protection in resetAndKeepLast agains empty vectors

Warn about empty collimator or env.vars object.
Protection in resetAndKeepLast agains empty vectors
@shahor02 shahor02 requested a review from a team as a code owner March 19, 2024 12:01
@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-apass
async-2023-pp-apass1
async-2022-pp-apass6
async-2022-pp-apass4
async-mc
async-data

@alibuild
Copy link
Collaborator

Error while checking build/O2/fullCI for 83f57da at 2024-03-20 02:50:

## sw/BUILD/O2-latest/log
c++: error: unrecognized command-line option '--rtlib=compiler-rt'
c++: error: unrecognized command-line option '--rtlib=compiler-rt'


## sw/BUILD/O2-full-system-test-latest/log
task timeout reached .. killing all processes


## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -v clang-diagnostic-error error-log.txt
++ grep ' error:'
++ GRERR=1
++ [[ 1 == 0 ]]
++ mkdir -p /sw/INSTALLROOT/dd4bd6ae5c65bd4211560d30795a6a0d959a647f/slc8_x86-64/o2checkcode/1.0-local249/etc/modulefiles
++ cat
--

Full log here.

@shahor02 shahor02 merged commit ed962c5 into AliceO2Group:dev Mar 20, 2024
@shahor02 shahor02 deleted the pr_dcsgrp branch March 20, 2024 08:20
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