Ticket #5684 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Extend IDF Logfile syntax

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: major Milestone: Release 2.2
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

Extend vocabulary of options to extract-single-value-as to include selected options from the Kernel/Statistics.h header. The ability to select the last log value is very important!

Keep the old mean/period n syntax for now, but will retire this in the future.

Share existing code where possible. The run object also performs this kind of filtering.

Also will need to update the IDF documentation.

Attachments

DemoParamFile.xml (312 bytes) - added by Owen Arnold 8 years ago.

Change History

comment:1 Changed 8 years ago by Owen Arnold

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Owen Arnold

refs #5684. Extend options.

Add tests and refactor existing code to make filtering a common free function.

Changeset: e4d7a3995eec7efed9291e7e6cd967017ce45fb3

comment:3 Changed 8 years ago by Owen Arnold

refs #5684. Fix error in cmakelists.txt

Changeset: d4cf7faba2ee9f3e1929af097a9673d758758f1d

comment:4 Changed 8 years ago by Owen Arnold

refs #5684. Fix warnings in test class

Changeset: c95595b71fcda12a9363d37639468e28292e45ab

comment:5 Changed 8 years ago by Owen Arnold

tester:

This is working now. This snippet and the attached file will help you test out the different options.

group = Load(Filename='csp79590.raw')
LoadParameterFile(Workspace=group, Filename="DemoParamFile.xml")
ws = group[0]
timeseries = ws.getRun().getLogData('theta')
first = timeseries.nthValue(0)
#last = timeseries.nthValue(timeseries.size()-1)

inst = ws.getInstrument()
holder = inst.getComponentByName('some-surface-holder')
pos = holder.getPos()

# the first value and pos.getZ() should be the same!!!!
print first
print pos.getZ() 

You can now modify the extract-single-value-as in the xml file to be any of the allowed option. You can then extract the corresponding values from the theta time series manually (see the commented last for example) See the IDF wiki page for all the options.

Changed 8 years ago by Owen Arnold

comment:6 Changed 8 years ago by Owen Arnold

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:7 Changed 8 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:8 Changed 8 years ago by Michael Reuter

  • Status changed from verifying to closed

This looks like it works just fine.

comment:9 Changed 8 years ago by Owen Arnold

refs #5684. Extend options.

Add tests and refactor existing code to make filtering a common free function.

Changeset: e4d7a3995eec7efed9291e7e6cd967017ce45fb3

comment:10 Changed 8 years ago by Owen Arnold

refs #5684. Fix error in cmakelists.txt

Changeset: d4cf7faba2ee9f3e1929af097a9673d758758f1d

comment:11 Changed 8 years ago by Owen Arnold

refs #5684. Fix warnings in test class

Changeset: c95595b71fcda12a9363d37639468e28292e45ab

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6530

Note: See TracTickets for help on using tickets.