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

[GCC9] Fix fortran warning in GeneratorInterface/PartonShowerVeto #28928

Closed
mrodozov opened this issue Feb 12, 2020 · 9 comments · Fixed by #29757
Closed

[GCC9] Fix fortran warning in GeneratorInterface/PartonShowerVeto #28928

mrodozov opened this issue Feb 12, 2020 · 9 comments · Fixed by #29757

Comments

@mrodozov
Copy link
Contributor

There is a warning about uninit var in:
https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc7_amd64_gcc900/CMSSW_11_1_X_2020-02-11-2300/GeneratorInterface/PartonShowerVeto
which we'd like to know what init value to use for, to fix the warning.

@cmsbuild
Copy link
Contributor

A new Issue was created by @mrodozov Mircho Rodozov.

@Dr15Jones, @smuzaffar, @silviodonato, @makortel, @davidlange6, @fabiocos can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@makortel
Copy link
Contributor

assign generators

@cmsbuild
Copy link
Contributor

New categories assigned: generators

@alberto-sanchez,@SiewYan,@qliphy,@efeyazgan,@mkirsano,@agrohsje you have been requested to review this Pull request/Issue and eventually sign? Thanks

@smuzaffar smuzaffar changed the title Fix fortran warning in GeneratorInterface/PartonShowerVeto [GCC9] Fix fortran warning in GeneratorInterface/PartonShowerVeto Feb 12, 2020
@agrohsje
Copy link

Just to say: I am not aware of anybody using this code.
YCUT should be something say 100 or similar. So we can initialize it with a neg value and throw an error if it is below 0.

@makortel
Copy link
Contributor

Just to say: I am not aware of anybody using this code.

Can this piece of code be removed then?

@silviodonato
Copy link
Contributor

Just to say: I am not aware of anybody using this code.

Can this piece of code be removed then?

@agrohsje

@agrohsje
Copy link

@qliphy @mkirsano @SiewYan
checking the history, I don't think this is still used:
https://github.com/cms-sw/cmssw/tree/master/GeneratorInterface/PartonShowerVeto
plotting matching is in genproductions. please comment as well.

@qliphy
Copy link
Contributor

qliphy commented Apr 23, 2020

It might still be safe to keep the package for the moment.

To fix the warning, I agree with @agrohsje "YCUT should be something say 100 or similar. So we can initialize it with a neg value and throw an error if it is below 0."

@smuzaffar
Copy link
Contributor

#29757 should resolve this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants