Ticket #9637 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

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:1 Changed 6 years ago by Michael Reuter

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Michael Reuter

  • Status changed from assigned to inprogress

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:4 Changed 6 years ago by Michael Reuter

  • Description modified (diff)

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

Note: See TracTickets for help on using tickets.