Ticket #2101 (closed)

Opened 10 years ago

Last modified 5 years ago

Scripts generated from some algorithms with complex string properties do not function correctly

Reported by: Martyn Gigg Owned by: Roman Tolchenov
Priority: major Milestone: Iteration 30
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

An example of this is running the IDA interface which does some fitting with a resolution function. One of the parameters within the function string is a filename that must be quoted. The problem is that within the Python script it is already inside quotes and so the quotes need to be escaped.

Change History

comment:1 Changed 10 years ago by Nick Draper

  • Owner set to Roman Tolchenov
  • Status changed from new to assigned

comment:2 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 27 to Iteration 28

Bulk move of tickets at the end of iteration 27

comment:3 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 28 to Iteration 29

Bulk move of tickets at the end of iteration 28

comment:4 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 29 to Iteration 30

Accepted and assigned tickets moved at iteration 29 code freeze

comment:5 Changed 9 years ago by Roman Tolchenov

  • Status changed from assigned to accepted

comment:6 Changed 9 years ago by Roman Tolchenov

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

In [13087]:

Changed double quotes to single quotes in script generation. This allows using double quotes in property values without breaking python. fixes #2101

comment:7 Changed 9 years ago by Nick Draper

  • Status changed from closed to verify
  • Resolution fixed deleted

Corrected status to verify

comment:8 Changed 9 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:9 Changed 9 years ago by Martyn Gigg

  • Status changed from verifying to closed

Verified with the Fit outputs from the indirect data analysis interface.

The Python strings now use single quotes so this should mitigate any problem with C strings.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2948

Note: See TracTickets for help on using tickets.