Ticket #7618 (closed: duplicate)

Opened 7 years ago

Last modified 5 years ago

Translator from FullProf to Mantid for initial fitting param starting value stage 2

Reported by: Anders Markvardsen Owned by: Karl Palmen
Priority: major Milestone: Release 3.2
Component: Tools Keywords:
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

Extend algorithm in #7617 to accept an InOutWorkspace, where if it is defined the FullProf parameters are applied to this workspace.

Here also make the OutputFile optional if InOutWorkspace is defined

Change History

comment:1 Changed 7 years ago by Anders Markvardsen

Wiki page which goes into some detail about have to convert parameters for the specific case of the Ikeda-Carpenter-Pseudo-Voigt: http://www.mantidproject.org/CreateIkedaCarpenterParameters

The wiki for Ikeda-Carpenter-Pseudo-Voigt is http://www.mantidproject.org/IkedaCarpenterPV

comment:2 Changed 7 years ago by Karl Palmen

I think such an algorithm could be called something like LoadFullprofToInstrumentParameters, Load... because it takes some data from a file and puts it into a workspace.

comment:3 Changed 7 years ago by Karl Palmen

  • Milestone changed from Release 3.0 to Backlog

Moved to backlog as agreed with Anders.

comment:4 Changed 7 years ago by Karl Palmen

  • Milestone changed from Backlog to Release 3.0

comment:5 Changed 7 years ago by Karl Palmen

It could be implemented as an expansion of LoadFullprofResolution.

comment:6 Changed 7 years ago by Karl Palmen

The simplest solution would be to expand ConvertFullprofToXML, because there we the Poco::DOM which can then be stuffed into the workspace via InstrumentDefinitionParser.setComponentLinks and workspace.popultateInstrumentParameters.

If this were done, we'd have a load algorithm and the name should be changed appropriately.

Alternatively, the code that creates the Poco::DOM could be put into a shared class. This shared class might be needed by other tickets in this area and a talk with the scientists is needed to determine these tickets.

comment:7 Changed 7 years ago by Karl Palmen

  • Milestone changed from Release 3.0 to Backlog

Moved to backlog, because the best approach to this ticket could be determined by future ctalks with scientists.

comment:8 Changed 7 years ago by Karl Palmen

  • Status changed from new to verify
  • Resolution set to duplicate

This is now done by LoadFullprofResolution.

comment:9 Changed 7 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:10 Changed 7 years ago by Anders Markvardsen

  • Status changed from verifying to closed

As stated this is now done by loadfullprofresolution

comment:11 Changed 7 years ago by Nick Draper

  • Milestone changed from Backlog to Release 3.2

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8463

Note: See TracTickets for help on using tickets.