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

LoadMD + FileBackEnd: error messages when ticking combinations of options #12442

Closed
mantid-builder opened this issue Apr 21, 2015 · 0 comments · Fixed by #13192
Closed

LoadMD + FileBackEnd: error messages when ticking combinations of options #12442

mantid-builder opened this issue Apr 21, 2015 · 0 comments · Fixed by #13192
Assignees
Labels
Framework Issues and pull requests related to components in the Framework Low Priority Things that you don't ever want to be done.

Comments

@mantid-builder
Copy link
Collaborator

This issue was originally TRAC 11604

Original Reporter: @FedeMPouzols

This is really minor, just noting it down here. It could be fixed when any other changes/additions are done in LoadMD, FileBackEnd and friends.

To test the FileBackEnd property of the algorithm LoadMD, I'm going to load the file
SEQ_MDEW.nxs (available in the system tests data).

  • If I select FileBackEnd and BoxStructureOnly but not MetadataOnly, an error (exception) will tell me: Both BoxStructureOnly and fileBackEnd were set to TRUE: this is not possible. That is correct. You can do this with the algorithm dialog or:
Load('SEQ_MDEW.nxs', OutputWorkspace='tws', FileBackEnd=1, BoxStructureOnly=1)
  • But if I select FileBackEnd and MetadataOnly, but not BoxStructureOnly, the same throw will tell me the same Both BoxStructureOnly and fileBackEnd were set to TRUE: this is not possible., even though I didn't selected BoxStructureOnly.
Load('SEQ_MDEW.nxs', OutputWorkspace='tws', FileBackEnd=1, MetaDataOnly=1)

It will tell you: Both BoxStructureOnly and fileBackEnd were set to TRUE: this is not possible.

This error message is not correct and could be confusing.

This seems to come down to how m_BoxStructureAndMethadata is used in the if that throws that exception.

@mantid-builder mantid-builder added Low Priority Things that you don't ever want to be done. Framework Issues and pull requests related to components in the Framework labels Jun 3, 2015
shahrozahmed1 pushed a commit that referenced this issue Jul 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framework Issues and pull requests related to components in the Framework Low Priority Things that you don't ever want to be done.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants