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

SANS support reduction in event mode #9349

Closed
1 task done
mantid-builder opened this issue Nov 22, 2013 · 2 comments
Closed
1 task done

SANS support reduction in event mode #9349

mantid-builder opened this issue Nov 22, 2013 · 2 comments
Labels
SANS Issues and pull requests related to SANS
Milestone

Comments

@mantid-builder
Copy link
Collaborator

Original Reporter: Gesner Passos

This ticket is blocked by :

This ticket is blocks : TRAC8366, TRAC8528

Larmor has the plan to collect data only in event mode. Besides, SANS2D has been collecting data in event mode, but the SANS interface uses only the histogram data inside
it to perform the reduction.

The goal of this ticket is to use the data in event mode ensuring that the result is compatible with the current one that uses histogram data.

@mantid-builder
Copy link
Collaborator Author

This issue was originally trac ticket 8505

@mantid-builder
Copy link
Collaborator Author

http://trac.mantidproject.org/mantid/raw-attachment/ticket/8505/compare_hist_event.py
(uploaded by Gesner Passos on 2013-11-29T09:12:05)


http://trac.mantidproject.org/mantid/raw-attachment/ticket/8505/check_gui_batchmode.csv
(uploaded by Gesner Passos on 2013-11-29T09:12:53)


@mantid-builder mantid-builder added the SANS Issues and pull requests related to SANS label Jun 3, 2015
@mantid-builder mantid-builder added this to the Release 3.1 milestone Jun 3, 2015
PeterParker added a commit that referenced this issue Jun 16, 2015
... to allow for a region of interest to be used for the transmission
rather than a monitor.
PeterParker added a commit that referenced this issue Jun 16, 2015
Get rid of the mess to do with the property, getter and setter.  It was
uncessary.
PeterParker added a commit that referenced this issue Jun 16, 2015
Includes addition to the SANSUtility unit tests, which apparently were
never being run as the file was not listed in CMakeList.txt.
PeterParker added a commit that referenced this issue Jun 16, 2015
Everything should be in place now.
PeterParker added a commit that referenced this issue Jun 16, 2015
Sorry for the double checkbuild.
PeterParker added a commit that referenced this issue Jun 16, 2015
The variable was never used unitialised, but initialising it all the same.
AntonPiccardoSelg added a commit that referenced this issue Jun 22, 2015
AntonPiccardoSelg pushed a commit that referenced this issue Sep 29, 2015
... to allow for a region of interest to be used for the transmission
rather than a monitor.
AntonPiccardoSelg pushed a commit that referenced this issue Sep 29, 2015
Get rid of the mess to do with the property, getter and setter.  It was
uncessary.
AntonPiccardoSelg pushed a commit that referenced this issue Sep 29, 2015
Includes addition to the SANSUtility unit tests, which apparently were
never being run as the file was not listed in CMakeList.txt.
AntonPiccardoSelg pushed a commit that referenced this issue Sep 29, 2015
Everything should be in place now.
AntonPiccardoSelg pushed a commit that referenced this issue Sep 29, 2015
It's now clear why these tests were never ran.  I am removing them but
will open up a new ticket to reinstate them at a later date.  We're not
losing anything here, since they were never run to begin with.
AntonPiccardoSelg pushed a commit that referenced this issue Sep 29, 2015
Sorry for the double checkbuild.
AntonPiccardoSelg pushed a commit that referenced this issue Sep 29, 2015
The variable was never used unitialised, but initialising it all the same.
martyngigg added a commit that referenced this issue Sep 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SANS Issues and pull requests related to SANS
Projects
None yet
Development

No branches or pull requests

1 participant