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

Refs #11244 fix xml output file #455

Merged
merged 1 commit into from Mar 26, 2015
Merged

Conversation

VickieLynch
Copy link
Contributor

XML output works in LoadParameterFile now;

    LoadIsawPeaks(Filename='MANDI_801.peaks', OutputWorkspace='peaks')
    LoadIsawUB(InputWorkspace='peaks', Filename='MANDI_801.mat')
    SCDCalibratePanels(PeakWorkspace='peaks',DetCalFilename='mandi_801.DetCal',XmlFilename='mandi_801.xml',a=74,b=74.5,c=99.9,alpha=90,beta=90,gamma=60)
    Load(Filename='MANDI_801_event.nxs', OutputWorkspace='MANDI_801_event')
    CloneWorkspace(InputWorkspace='MANDI_801_event', OutputWorkspace='MANDI_801_event_xml')
    LoadParameterFile(Workspace='MANDI_801_event_xml', Filename='mandi_801.xml')
    RenameWorkspace(InputWorkspace='MANDI_801_event_xml', OutputWorkspace='MANDI_801_event_DetCal')
    LoadIsawDetCal(InputWorkspace='MANDI_801_event_DetCal', Filename='mandi_801.DetCal')

~

@VickieLynch VickieLynch added Diffraction Issues and pull requests related to diffraction In Progress Single Crystal Issues and pull requests related to single crystal labels Mar 24, 2015
@VickieLynch VickieLynch added this to the Release 3.4 milestone Mar 24, 2015
@FedeMPouzols FedeMPouzols self-assigned this Mar 26, 2015
@FedeMPouzols
Copy link
Contributor

This fixes issue #11244.

@FedeMPouzols
Copy link
Contributor

All looks good, this seems to fix the issue and the modified test script posted on the trac issue worked well for me locally (tested on debian).

For the future it would be great if the example script included in the .rst file is turned into a doc-test, so we protect this functionality. For that doc-test the modified/simplified version posted on trac is probably better/faster, as it only uses the two small .peaks and .mat files.

FedeMPouzols added a commit that referenced this pull request Mar 26, 2015
@FedeMPouzols FedeMPouzols merged commit 3964efd into master Mar 26, 2015
@FedeMPouzols FedeMPouzols deleted the 11244_output_calibrate_scd branch March 26, 2015 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Diffraction Issues and pull requests related to diffraction Single Crystal Issues and pull requests related to single crystal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants