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

LoadVesuvio does not work with NeXus files #18843

Closed
martyngigg opened this issue Feb 14, 2017 · 0 comments · Fixed by #18884
Closed

LoadVesuvio does not work with NeXus files #18843

martyngigg opened this issue Feb 14, 2017 · 0 comments · Fixed by #18884
Assignees
Labels
Indirect/Inelastic Issues and pull requests related to indirect or inelastic Patch Candidate Urgent issues that must be included in a patch following a release
Milestone

Comments

@martyngigg
Copy link
Member

Bug report from Giovanni:

Actual behavior

I was trying to use "LoadVesuvio" to load the first calibration runs we could measure this evening, but I am getting the following strange error:

Error in u_len value read from file, it would be -26; where it is calculated as u_len = ad_data - ad_user - 2, where ad_data: 0, ad_user: 24
Error in execution of algorithm LoadVesuvio:
Inconsistent value for the field u_len found
at line 146 in 'C:/MantidInstall/plugins/python/algorithms\LoadVesuvio.py'
caused by line 232 in 'C:/MantidInstall/plugins/python/algorithms\LoadVesuvio.py'
caused by line 558 in 'C:/MantidInstall/plugins/python/algorithms\LoadVesuvio.py'
caused by line 609 in 'C:/MantidInstall/plugins/python/algorithms\LoadVesuvio.py'
caused by line 926 in 'C:\MantidInstall\bin\mantid\simpleapi.py'

My guess is that this has to do with the change in the names of the VESUVIO runs from EVS***** to VESUVIO********. However, the "Load" command works normally..

Could you please have a look. The last run I was trying to load is 25999.

Platforms affected

All

@martyngigg martyngigg added the Indirect/Inelastic Issues and pull requests related to indirect or inelastic label Feb 14, 2017
@martyngigg martyngigg added this to the Release 3.10 milestone Feb 14, 2017
@martyngigg martyngigg self-assigned this Feb 14, 2017
@martyngigg martyngigg added the Patch Candidate Urgent issues that must be included in a patch following a release label Feb 14, 2017
martyngigg added a commit that referenced this issue Feb 16, 2017
It also bases the instrument prefix on the run number being loaded.
Refs #18843
martyngigg added a commit that referenced this issue Feb 16, 2017
martyngigg added a commit that referenced this issue Feb 16, 2017
martyngigg added a commit that referenced this issue Feb 21, 2017
martyngigg added a commit that referenced this issue Feb 22, 2017
martyngigg added a commit that referenced this issue Feb 23, 2017
It also bases the instrument prefix on the run number being loaded.
Refs #18843

(cherry picked from commit 60a878c)

Disable logging of delete workspace coperations

Refs #18843

(cherry picked from commit 540f561)

Remove unused variable

Refs #18843

(cherry picked from commit a394e3b)

Fix LoadVesuvio run finder when extension is provided.

Refs #18843

(cherry picked from commit ce56023)

Only pass strings to LoadVesuvio

Refs #18843

(cherry picked from commit 339d13f)

Revert config changes after each documentation test

Refs #18843

(cherry picked from commit 6f4cee4)

Fix loader in case path contains "-" character

Refs #18843

(cherry picked from commit 41f8add)

Fix full path handling properly

Refs #18843

(cherry picked from commit 6c508da)

Add patch release note
martyngigg added a commit that referenced this issue Feb 23, 2017
LoadVesuvio will not assume it uses RAW files

It also bases the instrument prefix on the run number being loaded.
Refs #18843

(cherry picked from commit 60a878c)

Disable logging of delete workspace coperations

Refs #18843

(cherry picked from commit 540f561)

Remove unused variable

Refs #18843

(cherry picked from commit a394e3b)

Fix LoadVesuvio run finder when extension is provided.

Refs #18843

(cherry picked from commit ce56023)

Only pass strings to LoadVesuvio

Refs #18843

(cherry picked from commit 339d13f)

Revert config changes after each documentation test

Refs #18843

(cherry picked from commit 6f4cee4)

Fix loader in case path contains "-" character

Refs #18843

(cherry picked from commit 41f8add)

Fix full path handling properly

Refs #18843

(cherry picked from commit 6c508da)

Add patch release note
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Indirect/Inelastic Issues and pull requests related to indirect or inelastic Patch Candidate Urgent issues that must be included in a patch following a release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant