Ticket #7891 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Expose moment calculations to python

Reported by: Peter Peterson Owned by: Martyn Gigg
Priority: critical Milestone: Release 3.1
Component: Python Keywords:
Cc: Blocked By: #7672
Blocking: Tester: Roman Tolchenov

Description

Now that the calculations exist in C++, it would be good to expose them to python. Unfortunately the statistics class is not already in python to add this to.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 3.0 to Backlog

Moved out to backlog as not enough time in R3.0

comment:2 Changed 7 years ago by Martyn Gigg

  • Milestone changed from Backlog to Release 3.1

comment:3 Changed 7 years ago by Martyn Gigg

  • Priority changed from major to critical

comment:4 Changed 7 years ago by Martyn Gigg

  • Status changed from new to inprogress

Start exporting basic Statistics stuff to Python.

Refs #7891

Changeset: 8690f39236e70e59b38b8742056abd6254ccc7f1

comment:5 Changed 7 years ago by Martyn Gigg

Add export for getMomentsAboutOrigin

Refs #7891

Changeset: 7d8f1e4016c599d64bbc80e26dae14ad726807ae

comment:6 Changed 7 years ago by Martyn Gigg

Let's just support Python float input for now.

Refs #7891

Changeset: dc61a5661d7a227b67377874902f0898261c198a

comment:7 Changed 7 years ago by Martyn Gigg

Export getMomentsAboutMean function

Refs #7891

Changeset: 96f4b4bd5e1ac7dcf09584aae246e67a3d620137

comment:8 Changed 7 years ago by Martyn Gigg

Refactor to avoid duplicated code in moments proxy functions.

Refs #7891

Changeset: 1feb77f6f0af9a3771d2937645827fe6c681c215

comment:9 Changed 7 years ago by Martyn Gigg

Python 2.6 doesn't have the delta argument for some asserts.

Replace here with more long winded comparison that does the same thing by hand. Refs #7891

Changeset: 518bf172c8a88f23379526cb32416c71a0b4b5f9

comment:10 Changed 7 years ago by Martyn Gigg

Add Z score functions to Stats namespace.

Refs #7891

Changeset: 6adb3a23379c0dcd624010638963ce8f38871f78

comment:11 Changed 7 years ago by Martyn Gigg

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

Branch: feature/7891_statistics_export_python

Tester: I advise looking at the StatisticsTest.py for examples of usage and then playing with the functions. Check that all interaction happens through numpy arrays.

comment:12 Changed 7 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:13 Changed 7 years ago by Roman Tolchenov

  • Status changed from verifying to verify
  • Tester Roman Tolchenov deleted

The changes are not on develop yet.

comment:14 Changed 7 years ago by Martyn Gigg

Yes they are. 6adb3a23379c0dcd624010638963ce8f38871f78 shows that the commit and hence the parents are all on develop + feature/7891_statistics_export_python

comment:15 Changed 7 years ago by Roman Tolchenov

I meant the binaries didn't include them yet

comment:16 Changed 7 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:17 Changed 7 years ago by Roman Tolchenov

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/7891_statistics_export_python'

Full changeset: cbb02751468c4a5d4d71804cc3e33c4bf82a43e3

comment:18 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8736

Note: See TracTickets for help on using tickets.