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

System test ReflectometryReductionOneAuto with OFFSPEC #529

Merged
merged 3 commits into from Apr 8, 2015

Conversation

eXeC64
Copy link
Contributor

@eXeC64 eXeC64 commented Apr 7, 2015

This is for trac ticket #10847

This pull request adds a systemtest that verifies that ReflectometryReductionOneAuto processes OFFSPEC data correctly.

Testing:

  • Review changes
  • Verify the new systemtest passes

@eXeC64 eXeC64 added Reflectometry Issues and pull requests related to reflectometry System tests labels Apr 7, 2015
@eXeC64 eXeC64 added this to the Release 3.4 milestone Apr 7, 2015
@DanNixon
Copy link
Member

DanNixon commented Apr 8, 2015

I don't know how important it actually is since the test seems to be fine with it but there is a specific folder for reference results to be kept separate from data (SystemTests/tests/analysis/reference).

@eXeC64
Copy link
Contributor Author

eXeC64 commented Apr 8, 2015

So there is, I didn't notice it. I'll move the reference result across.

@DanNixon DanNixon self-assigned this Apr 8, 2015
DanNixon added a commit that referenced this pull request Apr 8, 2015
…eauto_offspec

System test ReflectometryReductionOneAuto with OFFSPEC
@DanNixon DanNixon merged commit 06c19a4 into master Apr 8, 2015
@DanNixon DanNixon deleted the 10847_systemtest_reflredoneauto_offspec branch April 8, 2015 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Reflectometry Issues and pull requests related to reflectometry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants