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.

Sign up for GitHub

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

Jump to bottom

Angle summing - remove "accumulate" option #6

Closed
mducle opened this issue Jan 16, 2024 · 0 comments · Fixed by #8
Closed

Angle summing - remove "accumulate" option #6

mducle opened this issue Jan 16, 2024 · 0 comments · Fixed by #8

Comments

@mducle
Copy link
Member

mducle commented Jan 16, 2024

The accumulate option is confusing to use - there should just two options (ignore and replace).
mducle added a commit that referenced this issue Jan 25, 2024
@mducle
Remove 'accumulate' angle summing option #6
c7bdcc2
@mducle mducle mentioned this issue Jan 25, 2024
Merged
mducle added a commit that referenced this issue Jan 25, 2024
@mducle
Cycle 23/4 bugs (#8)
1e0c7fe 
* Fix simple bugs from Cycle 23/4

Fix auto-ei for updatestored runs
Auto-ei now updates for each run if not using monovan
sample_bg now correctly NormaliseByCurrent'd
keepworkspaces reverted to False by default

* Re-enable instrument copying function

Fix issue with reading powder nxspe files in Mantid
  (missing NXClass in detector group)
Add extra field requested by Toby
Add handling for MARI data before Ei_nominal field introduced.

* Add numerical verification

* Remove 'accumulate' angle summing option #6
@mducle mducle closed this as completed in #8 Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Cycle234bugs mantidproject/direct_reduction
1 participant
@mducle