Ticket #10310 (closed)

Opened 6 years ago

Last modified 5 years ago

ExportExperimentLog overwrites dictionary key for logname

Reported by: Jose Borreguero Owned by: Wenduo Zhou
Priority: critical Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Jose Borreguero

Description

ExportExperimentLog( ... SampleLogNames="SensorA,SensorA,SensorA", SampleLogOperation="average,min,max", ...) will print max value for three requests for SensorA because self._sampleLogOperations is a dictionary whose 'SensorA' key is overwritten multiple times, instead of having three entries.

Change History

comment:1 Changed 6 years ago by Wenduo Zhou

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Wenduo Zhou

  • Status changed from assigned to inprogress

comment:3 Changed 6 years ago by Wenduo Zhou

Refs #10310. Fixed the operation issue.

On branch bugfix/10310_log_operation

modified: ../Mantid/Framework/PythonInterface/plugins/algorithms/ExportExperimentLog.py

Changeset: dba4b4eed3b8b6d8c0a3892d5fb7e5c949a9078e

comment:4 Changed 6 years ago by Wenduo Zhou

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

For tester

Run the script as 'ExportExperimentLog( ... SampleLogNames="SensorA,SensorA,SensorA", SampleLogOperation="average,min,max", ...)'.

comment:5 Changed 6 years ago by Jose Borreguero

  • Status changed from verify to verifying
  • Tester set to Jose Borreguero

comment:6 Changed 6 years ago by Jose Borreguero

  • Status changed from verifying to reopened
  • Resolution fixed deleted

comment:7 Changed 6 years ago by Wenduo Zhou

  • Status changed from reopened to inprogress

Refs #10310. Fixed the bug.

Changeset: 5d2340778a56caf7d824b4dcf94bd38cc16f6c58

comment:8 Changed 6 years ago by Jose Borreguero

  • Status changed from inprogress to closed

Merge remote-tracking branch 'origin/bugfix/10310_log_operation'

Full changeset: ​9fa7e384d1227b6d39f24b39921ba8e22912990a

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11152

Note: See TracTickets for help on using tickets.