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

Used geant4 10.7 hadronic verbosity control #645

Merged
merged 1 commit into from
Nov 4, 2021
Merged

Conversation

resnegfk
Copy link
Contributor

@resnegfk resnegfk commented Nov 4, 2021

Replaced older hadronic verbosity control with the newer one (in the remaining file used in the MT mode)

@FNALbuild
Copy link
Collaborator

Hi @resnegfk,
You have proposed changes to files in these packages:

  • Mu2eG4

which require these tests: build.

@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.

The following users requested to be notified about changes to these packages:
@resnegfk

⌛ The following tests have been triggered for 73c23f4: build (Build queue has 1 jobs)

About FNALbuild. Code review on Mu2e/Offline.

@FNALbuild
Copy link
Collaborator

☀️ The tests passed at 73c23f4.

Test Result Details
merge Merged 73c23f4 at 4ba81da
build (prof) Log file. Build time: 14 min 05 sec
ceSimReco Log file.
g4test_03MT Log file.
transportOnly Log file.
POT Log file.
g4study Log file.
cosmicSimReco Log file.
cosmicOffSpill Log file.
ceSteps Log file.
ceDigi Log file.
muDauSteps Log file.
ceMix Log file.
rootOverlaps Log file.
g4surfaceCheck Log file.
FIXME, TODO TODO (0) FIXME (0) in 1 files
clang-tidy 〰️ 0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at 73c23f4 after being merged into the base branch at 4ba81da.

For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.

@brownd1978 brownd1978 self-assigned this Nov 4, 2021
Copy link
Collaborator

@kutschke kutschke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this change reduce printout? Increase it? Provide new ways to let us adjust the verbosity at run time?

@resnegfk
Copy link
Contributor Author

resnegfk commented Nov 4, 2021

It will decrease some printout (which Ray noted).
We have been using this approach in the sequential mode (and master thread) for a while now.
The PR makes a change missed earlier.

@FNALbuild
Copy link
Collaborator

📝 The HEAD of main has changed to 3056dc1. Tests are now out of date.

@kutschke
Copy link
Collaborator

kutschke commented Nov 4, 2021

Thanks for the explanation.

@kutschke kutschke merged commit 40921f4 into Mu2e:main Nov 4, 2021
@resnegfk resnegfk deleted the g4hpv branch January 29, 2022 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants