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

Update UTM library 0.9.0 for hadronic showers in GT #946

Open
rekovic opened this issue Sep 8, 2021 · 8 comments · Fixed by cms-sw/cmsdist#7281
Open

Update UTM library 0.9.0 for hadronic showers in GT #946

rekovic opened this issue Sep 8, 2021 · 8 comments · Fixed by cms-sw/cmsdist#7281

Comments

@rekovic
Copy link

rekovic commented Sep 8, 2021

Request to update the UTM library in externals using the latest version utm_0.9.0 which supports the hadronic showers grammar for uGT.

Needed in master and 12_0_X.

The code is available:
https://gitlab.cern.ch/cms-l1t-utm/utm/-/tree/utm_0.9.0

Attn @smuzaffar @mrodozov

@rekovic rekovic changed the title Update UTM library for hadronic showers in GT Update UTM library 0.9.0 for hadronic showers in GT Sep 8, 2021
mrodozov added a commit to cms-sw/cmsdist that referenced this issue Sep 8, 2021
@mrodozov
Copy link

mrodozov commented Sep 8, 2021

@rekovic I'd like to use the occasion to let you know we have a report for undefined behavior coming from UTM
should I explain more here or open an issue on gitlab@cern ?

@smuzaffar
Copy link

smuzaffar commented Sep 13, 2021

@rekovic , utm failed to build with make -j 16. Looks like parallel build is broken. @mrodozov has opened an issue https://gitlab.cern.ch/cms-l1t-utm/utm/-/issues/37 but there is no activity on that. I am afraid we have to wait for utm developers to fix this build issue first.

@dildick
Copy link

dildick commented Sep 17, 2021

@smuzaffar
Copy link

@dildick , we are testing 0.9.1 here cms-sw/cmsdist#7281

@dildick
Copy link

dildick commented Sep 17, 2021

Very good. Thanks for keeping me in the loop!

@dildick
Copy link

dildick commented Sep 20, 2021

Nice! UTM 0.9.1 was merged in CMSSW_12_1_X

@dildick
Copy link

dildick commented Sep 20, 2021

@rekovic Do we need to also backport it to CMSSW_12_0_X, so that we can work on a branch for the cms-l1t-offline software?

@dildick
Copy link

dildick commented Sep 28, 2021

@rekovic This was done for CMSSW_12_0_X and CMSSW_12_1_X, so the issue can be closed.

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 a pull request may close this issue.

4 participants