Ticket #7891 (closed: fixed)
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: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:6 Changed 7 years ago by Martyn Gigg
Let's just support Python float input for now.
Refs #7891
Changeset: dc61a5661d7a227b67377874902f0898261c198a
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
Moved out to backlog as not enough time in R3.0