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

Geant4 BFC options #613

Merged
merged 3 commits into from
Oct 30, 2023
Merged

Conversation

klendathu2k
Copy link
Contributor

Adding BFC options to load geant4 libraries, and load the relevant spack modules to support. (The latter is better added to the star-env modules?)

... collapse some of these options together

... remove options which are not likely to be used (we can always add them back if needed).
@klendathu2k klendathu2k marked this pull request as draft October 25, 2023 13:21
@klendathu2k
Copy link
Contributor Author

Not quite ready for prime time... seems to break several tests. Investigating.

@klendathu2k klendathu2k force-pushed the geant4-bfc-integration branch from 7028d8f to 28599e1 Compare October 25, 2023 19:06
@klendathu2k
Copy link
Contributor Author

In order to test functionality, need to load G4 modules via

$ module load geant4-10.4.0                  
$ module load geant4-data-10.4.0 # may not be required

@plexoos plexoos marked this pull request as ready for review October 26, 2023 19:47
@klendathu2k
Copy link
Contributor Author

Any thoughts as to why the whitespace in BigFullChain.h was causing runtime errors?

@plexoos
Copy link
Member

plexoos commented Oct 26, 2023

The white space change was used to trigger CI. There was a small adjustment in the container environment

Copy link
Contributor

@genevb genevb left a comment

Choose a reason for hiding this comment

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

No objections.

@klendathu2k
Copy link
Contributor Author

And... we merge.

@klendathu2k klendathu2k merged commit 040e204 into star-bnl:main Oct 30, 2023
148 checks passed
dkapukchyan pushed a commit to dkapukchyan/star-sw that referenced this pull request Mar 11, 2024
Adding BFC options to load geant4 libraries. 

---------

Co-authored-by: Dmitri Smirnov <[email protected]>
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.

3 participants