Ticket #7618 (closed: duplicate)
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: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: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:12 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 8463
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