Ticket #9637 (closed: fixed)
Add ability to create array properties in Python
Reported by: | Michael Reuter | Owned by: | Michael Reuter |
---|---|---|---|
Priority: | major | Milestone: | Release 3.2 |
Component: | Python | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Martyn Gigg |
Description (last modified by Michael Reuter) (diff)
We want to be able to do things like the following from Python:
pm = PropertyManager() pm["a"] = (1,2,3) pm["b"] = [1,2,3]
Change History
comment:3 Changed 6 years ago by Michael Reuter
Refs #9637. Adding logic for array property creation.
Makes everything double vector, but that gets fixed next.
Changeset: 85378dcdcf04835fbd5efb5d1cf778af559c9c95
comment:5 Changed 6 years ago by Michael Reuter
Refs #9637. Expanding array typing logic.
I had to remove the NumPy array stuff as this was giving the unit test fits.
Changeset: 6b94687b417d1731ccc84913703e036785febdfd
comment:6 Changed 6 years ago by Michael Reuter
Refs #9637. Adding a unit test. Doesn't work.
This is giving a boost::python::throw_error_already_set when try to run the tuple test. Sigh.
Changeset: 3315f296d20da361f0fbc9e9c388f295e135caed
comment:7 Changed 6 years ago by Michael Reuter
Refs #9637. Working unit test with new function template and macro.
Had to directly point to PySequenceToVector to get things wokring properly.
Changeset: 79ad81bbdb4e4bcc218aafb312f334f70864afdc
comment:8 Changed 6 years ago by Michael Reuter
Refs #9637. Reorder bool check due to casting issues.
Changeset: 2c6368ed11293fd860485a62f60c5a967b62248d
comment:9 Changed 6 years ago by Michael Reuter
Refs #9637. Adding more unit tests.
Changeset: ef7979d9683302928c19498d9cdf8243d4b2c465
comment:10 Changed 6 years ago by Michael Reuter
Refs #9637. Fix a doxygen mismatch.
Changeset: e9a497140c8ec18319d963ff98a0f9408cc71325
comment:11 Changed 6 years ago by Michael Reuter
- Status changed from inprogress to verify
- Resolution set to fixed
The branch can be found here.
To test: Check that the above examples work and that the unit test runs for PropertyWithValueFactory. NumPy arrays should not work if you want to try that.
comment:12 Changed 6 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester set to Martyn Gigg
comment:13 Changed 6 years ago by Peter Peterson
Seems to work for me.
comment:14 Changed 6 years ago by Martyn Gigg
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/9637_array_props_python'
Full changeset: fbe93866be813b42ac87d1f632321dd05e91a5d9
comment:15 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 10480