ets
ets copied to clipboard
Migrate on_trait_change to observe
A large amount of work has already been undertaken to move ETS projects from on_trait_change
to observe
. This includes replacing use of the static trait change handlers such as _*_changed
and the use of depends_on
in Property
traits.
We would like to migrate all of ETS if possible but given the time/resource constraints, we are going to go with the 90%-10% option here. Given our experience, we know that 90% of the work is more-or-less straightforward whereas 10% of the work leads us down rabbitholes which require significant debugging/investigation. Therefore, at the moment, we are only going to attack the easy/straightforward 90% and ignore the hard/difficult 10% of changes.
Note also that in some packages, this work was completed but a release hasn't been published so the changes aren't visible to the users. We need to expose the changes to the users in order to catch any issues potentially not caught by the testsuites.
- [ ]
depends_on
is used a lot, especially in the examples, static@on_trait_change
and dynamicon_trait_change
are used heavily and so are the_*_changed
handlers in Traits - [ ] There are still a large number of static change handlers i.e.
_*_changed
methods in PyFace. - [ ] There are uses of
property_depends_on
, dynamic uses ofon_trait_change
and a number of static change handlers i.e._*_changed
methods in TraitsUI - [ ] There are uses of the static
@on_trait_change
decorator and dynamic use ofon_trait_change
along with a few_*_changed
handlers in Envisage. - [ ] There are a few uses of dynamic
on_trait_change
and a few_*_changed
methods in AppTools - [ ] There are a few uses of dynamic
on_trait_change
and a large number of_*_changed
methods in Enable - [ ] There's one use of
depends_on
, a few uses ofon_trait_change
and a large number of_*_changed
methods in Chaco - [ ] SciMath hasn't been touched at all at the moment
Note : From the looks of it, we missed the _*_fired
special methods.