-
Notifications
You must be signed in to change notification settings - Fork 105
building hA2025 model #459
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a very incomplete look at a few of the files with some feedback comments
| @@ -0,0 +1,26 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be a private, specific script and I question it's general use and thus the addition to the repo
| @@ -0,0 +1,1631 @@ | |||
| //____________________________________________________________________________ | |||
| /* | |||
| Copyright (c) 2003-2024, The GENIE Collaboration | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copyrights everywhere should be brought up-to-date to include 2025
| For the full text of the license visit http://copyright.genie-mc.org | ||
|
|
||
|
|
||
| Author: Steve Dytman <[email protected]>, Pittsburgh Univ. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this author list accurate? The date at the end suggests that this is a from a copy of another file.
|
|
||
| For the class documentation see the corresponding header file. | ||
|
|
||
| Important revisions after version 2.0.0 : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these revision comments accurate for this file?
I suspect that the boilerplate comments for all the files in this PR need to be reviewed for relevancy and completeness.
adding a new class for hA2025 with the corresponding cross section data