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

SIREN ND280+ MODULE ADDITION #73

Merged
merged 44 commits into from
Aug 14, 2024
Merged

Conversation

mingshauliu
Copy link
Contributor

The density and material files are done and checked, I presented the Mathematica 'CAD' in GM but I can automate that too.

The injection uses high energy supernova in the deep-inelastic scattering example.

@mingshauliu
Copy link
Contributor Author

VISUAL INTERFACE CREATED FOR MATHEMATICA IN /resources/Detectors/visuals/

Copy link
Collaborator

@nickkamp1 nickkamp1 left a comment

Choose a reason for hiding this comment

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

@mingshauliu most of the changes look good! there are a few extraneous files that should be removed before merging (e.g., all the ".DS_Store" files and the "test_jul20.ipynb" file). Also could you revert the changes to the PaperPlots notebooks? I would like those to remain consistent with the paper. You can keep the DipolePortal_ND280UPGR.py file though, that will be useful for users

Copy link
Collaborator

@nickkamp1 nickkamp1 left a comment

Choose a reason for hiding this comment

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

After changes in above comment I will approve this PR. @austinschneider feel free to add any comments you might have though

@austinschneider
Copy link
Collaborator

I agree with @nickkamp1 's comments. As far as the extraneous files go, I think they either need to be removed from the history or we can do a squash merge to keep them out of the history.

@mingshauliu
Copy link
Contributor Author

Thank you both for the comments! I've deleted the extraneous files and reverted both PaperPlots, I'll request a review again. Let me know if anything else is needed!

@mingshauliu mingshauliu requested a review from nickkamp1 August 7, 2024 01:15
Copy link
Contributor Author

@mingshauliu mingshauliu left a comment

Choose a reason for hiding this comment

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

All checked!

Copy link
Collaborator

@nickkamp1 nickkamp1 left a comment

Choose a reason for hiding this comment

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

looks good to me know, approving! Thanks @mingshauliu

@nickkamp1
Copy link
Collaborator

I agree with @nickkamp1 's comments. As far as the extraneous files go, I think they either need to be removed from the history or we can do a squash merge to keep them out of the history.

Makes sense to me. Let's do a squash merge

@austinschneider austinschneider merged commit 4858ccc into Harvard-Neutrino:main Aug 14, 2024
31 checks passed
nickkamp1 pushed a commit that referenced this pull request Aug 23, 2024
* Dimension file of ND280 Upgrade

* Dimensions added

* ND280 Upgrade detector material file

* ND280 Upgrade component materials added

* Added rock

* space

* delete redundant comments

* ND28P UPGRD EVENT GENERATION

* CHANGED EXPERIMENT FILE

* Deep inelastic scattering within ND280 upgrade

* ISOTROPIC NEUTRINO INJECTION AND PLOT

* DIS plot for ND280+

* z shift of detector coordinates

* ND280+ PLOT UPDATE

* INJECTION GEOMETRY UPDATES

* EFFECTIVE AREA CALCULATION FOR ND280+

* NEUTRINO FLUX FOR T2K NEAR DETECTOR 2020

* ADDED FLUX CALCULATOR AND PROCESSED NU ENHANCED DATA

* NU ENHANCED DATA PROCESSED

* NEW FLUX CALCULATOR THAT RETURNS FILE W/O WRITING

* MOVE TO CORRECT DIRECTORY

* MOVE DIRECTORY

* JPARC NEUTRINO FLUX CALCULATOR EST. 2020

* RESTRUCTURED FLUX DATA

* CORRECTED FIDUCIAL VOL
CORRECTED FLUXCALCULATOR
EVENT GENERATION DENSITY PLOT

* FLUX CALCULATION FOR ND280+

* CREATED VISUAL INTERFACE FOR DETECTORS
IN MATHEMATICA

* CREATED VISUAL INTERFACE IN MATHEMATICA

* added read me

* added readme

* Update README.md

* Delete .DS_Store

* Delete resources/.DS_Store

* Delete resources/Detectors/.DS_Store

* Delete resources/Detectors/densities/.DS_Store

* Delete resources/Detectors/materials/.DS_Store

* Delete resources/Examples/.DS_Store

* Delete resources/Fluxes/.DS_Store

* Delete resources/Fluxes/T2K_NEAR/.DS_Store

* Delete notebooks/test_jul20.ipynb

* Delete resources/Examples/Example1/.DS_Store

* Revert PaperPlots for Example2

* Revert PaperPlots in Example1

* Delete resources/Fluxes/T2K_NEAR/T2K_NEAR-v1.0/.DS_Store
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