Ticket #6508 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Expose TimeSeriesProperty::valuesAsVector() to the Python interface

Reported by: Jose Borreguero Owned by: Martyn Gigg
Priority: major Milestone: Release 2.5
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Andrei Savici

Description

Exposing std::vector<TYPE> valuesAsVector() const; to the python interface should result in a python method returning a read-only tuple of TYPE objects

Change History

comment:1 Changed 8 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Martyn Gigg

Expose TimeSeriesProperty valueAsVector as clone nparray Refs #6508

Changeset: b2d514108cafd358a8ef65de290c00f5a299776b

comment:3 Changed 8 years ago by Martyn Gigg

Include required for some compilers. Refs #6508

Changeset: 96c6973615dd271227baefd10c13903f7986d285

comment:4 Changed 8 years ago by Martyn Gigg

Fix include of CloneToNumpy. Refs #6508

Changeset: cfce7b15cda2728eb95e21a451e6f2ddaefa8658

comment:5 Changed 8 years ago by Martyn Gigg

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

Tester:

  • Load a file that contains a time series log, suggestion=LOQ49886.nxs;
  • Right-click on the workspace when loaded and click show logs;
  • Find a time-series log, suggestion TEMP1

Check in python that the .value property of the log contains the expected number of entries and is a numpy array: 

run = ws.getRun()
log = run.getLogData("TEMP1")
values = log.value
print type(values)

comment:6 Changed 8 years ago by Russell Taylor

Re #6508. Temporarily remove test that's failing on Windows.

Changeset: 6fbb4b1dec51f477b3e06037a880741a8cfb94ad

comment:7 Changed 8 years ago by Russell Taylor

  • Status changed from verify to reopened
  • Resolution fixed deleted

Test failure on Windows needs addressing, and the test re-enabling.

comment:8 Changed 8 years ago by Martyn Gigg

Be less strict on element types Refs #6508

Windows dislikes the element type comparison even though it looks the same when you print them out...

Changeset: d2a724a94f812bdd281d905b2bbe02af0e0f9bf0

comment:9 Changed 8 years ago by Martyn Gigg

  • Status changed from reopened to accepted

comment:10 Changed 8 years ago by Martyn Gigg

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

comment:11 Changed 8 years ago by Andrei Savici

  • Status changed from verify to verifying
  • Tester set to Andrei Savici

comment:12 Changed 8 years ago by Andrei Savici

  • Status changed from verifying to closed

You can expose the times as numpy array as well

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7354

Note: See TracTickets for help on using tickets.