Skip to content

Conversation

@hayakawa16
Copy link
Member

Just preparing for review. Not ready yet.

… new branch because somehow (ugh!) I cleaned Kernel but still results in file I pushed. So needed to add other files I have on this branch.
@hayakawa16 hayakawa16 linked an issue Sep 11, 2025 that may be closed by this pull request
@hayakawa16
Copy link
Member Author

I'm ready for review.

Copy link
Member

@lmalenfant lmalenfant left a comment

Choose a reason for hiding this comment

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

I was able to run analog-vs-caw-efficiency.py

but fluence-vs-number-of-photons.py didn't plot anything, didn't show any data or error and returned the command prompt.

…-scripts-for-possible-short-course-use' into feature/25c-add-more-monte-carlo-scripts-for-possible-short-course-use
Copy link
Member

@lmalenfant lmalenfant left a comment

Choose a reason for hiding this comment

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

This looks good!

@hayakawa16 hayakawa16 merged commit 6bbbd42 into main Sep 18, 2025
@hayakawa16 hayakawa16 deleted the feature/25c-add-more-monte-carlo-scripts-for-possible-short-course-use branch September 18, 2025 20:56
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.

Add more monte carlo scripts for possible short course use

3 participants