Skip to content

Use less neutrons and remove sections where we increase number of neutrons#15

Merged
nvaytet merged 4 commits intomainfrom
less-neutrons
Jan 9, 2026
Merged

Use less neutrons and remove sections where we increase number of neutrons#15
nvaytet merged 4 commits intomainfrom
less-neutrons

Conversation

@nvaytet
Copy link
Member

@nvaytet nvaytet commented Jan 9, 2026

  • We replace the old versions of the mcstas notebooks with the short ones
  • Use less neutrons in the mcstas notebooks because we don't need so many statistics as we are not doing analysis

@nvaytet
Copy link
Member Author

nvaytet commented Jan 9, 2026

It all seems to run fine (in half the time!).
Only thing that appears to be broken is the QENS analysis, where the fit fails:
7ff46a68506ff2cd310eeac13d0e54ec94f04821289a9a5161df29a66ae1af51

But since we are not doing the analysis part, I'm fine with leaving it as is, and trying to fix it later.

@nvaytet nvaytet merged commit cb5b211 into main Jan 9, 2026
3 checks passed
@nvaytet nvaytet deleted the less-neutrons branch January 9, 2026 15:24
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.

1 participant