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

Add algorithm SaveSavuTomoConfig (save savu tomography reconstruction pipeline config) #311

Merged
merged 21 commits into from Feb 27, 2015

Conversation

FedeMPouzols
Copy link
Contributor

This addresses ticket #10766, and is a sibling of PR #286.

This algorithm saves a table workspace with 'tomography reconstruction pipeline' configuration/settings data into a file in the format used by the savu tool (https://github.com/DiamondLightSource/Savu).

Tester:

  • Review code.
  • The algorithm comes with a unit test and a doc test, make sure that those are fine.
  • You can also test it manually/interactively. With this (or similar to the doc test) you can generate a table as needed in the input to this algorithm:
t=CreateEmptyTableWorkspace()
t.addColumn("str", "ID")
t.addColumn("str", "Parameters")
t.addColumn("str", "Name")
t.addColumn("str", "Cite")

t.addRow(['save.plugin1', '{"silly param": 3}', 'Pipeline step 1', 'No cite'])
t.addRow([ 'save.median_filter', '{x: 0}', 'Pipeline plugin 2',  'Cite 2'])

The algorithm writes NeXus files like the example process??.nxs files found from here https://github.com/DiamondLightSource/Savu/tree/master/test_data.

@FedeMPouzols FedeMPouzols added this to the Release 3.4 milestone Feb 26, 2015
@FedeMPouzols FedeMPouzols added the Diffraction Issues and pull requests related to diffraction label Feb 26, 2015
@raquelalvarezbanos raquelalvarezbanos self-assigned this Feb 27, 2015
raquelalvarezbanos added a commit that referenced this pull request Feb 27, 2015
…nfig

Add algorithm SaveSavuTomoConfig (save savu tomography reconstruction pipeline config)
@raquelalvarezbanos raquelalvarezbanos merged commit cbc7d62 into master Feb 27, 2015
@raquelalvarezbanos raquelalvarezbanos deleted the feature/10766_save_tomo_config branch February 27, 2015 14:49
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants