Skip to content

Singularity v46#655

Merged
davidwalter2 merged 13 commits intoWMass:mainfrom
bendavid:singularity_v45
Feb 12, 2026
Merged

Singularity v46#655
davidwalter2 merged 13 commits intoWMass:mainfrom
bendavid:singularity_v45

Conversation

@bendavid
Copy link
Collaborator

@bendavid bendavid commented Feb 5, 2026

Major upgrade of singularity container to v46 (see https://gitlab.cern.ch/bendavid/cmswmassdocker/-/commits/v46?ref_type=tags)

Some small corresponding technical updates which are needed (migrate python tensorflow lite usage to new litert package)

Also fix a few array out of bounds accesses (off by one errors) which caused problems after the ROOT/cling/etc update (but were already bugs previously in principle)

Note that critically now we are back to a situation again where we cannot/should not import tensorflow before the event loop (but we no longer need to)

I will see if a reasonable solution can be found to this stripping off symbols, but it might also be ok like this. (if it happens by mistake narf will detect it and give a descriptive error rather than failing mysteriously/unpredictably)

@bendavid
Copy link
Collaborator Author

bendavid commented Feb 5, 2026

(CI will fail until the new container version propagates to cvmfs)

@bendavid bendavid force-pushed the singularity_v45 branch 6 times, most recently from ca3331a to 29eeddf Compare February 6, 2026 14:23
@bendavid
Copy link
Collaborator Author

bendavid commented Feb 7, 2026

From the CI, numerical results are 1:1 identical.

There are some minor aesthetic changes to the plots probably related to updated matplotlib, etc.

The only substantive issue I can see so far is that "Preliminary" has disappeared from the plots, probably due to the second line of text getting lost somehow.

Please others have a look as well though.

https://cmsmwbot.web.cern.ch/WMassAnalysis/PRValidation/PR655/2026_02_07/

@bendavid
Copy link
Collaborator Author

bendavid commented Feb 7, 2026

Ok the text subtitle issue is fixed. In principle this should be good to go now, but before it's merged first WMass/wums#15 needs to be merged and also we should figure out why the propagation of the updated container to cvmfs beyond CERN seems to be stuck/slow.

@bendavid
Copy link
Collaborator Author

bendavid commented Feb 8, 2026

Ok the propagation of CVMFS seems to be ok now (and the was merged) so this one can be merged now unless someone spots any other issues.

@bendavid bendavid changed the title Singularity v45 Singularity v46 Feb 10, 2026
@davidwalter2 davidwalter2 merged commit 441c5e9 into WMass:main Feb 12, 2026
19 checks passed
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