search

phetsims / axon

Axon provides powerful and concise models for interactive simulations, based on observable Properties and related patterns.
MIT License
10 stars 8 forks source link

Opt in to DerivedProperty instrumented linked elements #431

Closed zepumph closed 1 year ago

zepumph commented 1 year ago

Over in https://github.com/phetsims/scenery/issues/1541, we decided to make this feature an opt-in option, and only opt into it for PatternStringProperty.

After we implement this, we should take a hard look at all the linked elements we lose, and see if we want to opt in to any others.

zepumph commented 1 year ago

JUst kidding, we decided not to.

pixelzoom commented 1 year ago

Reopening. Didn't we decide that PatternStringProperty would opt-in to phetioLinkDependencies? All of our PatternStringProperty instances in calculus-grapher are suddently missing dependencies in the Studio tree.

This is blocking for Calculus Grapher dev testing, to begin 3/20. https://github.com/phetsims/calculus-grapher/issues/290

zepumph commented 1 year ago

There are no associated commits with this issue. We didn't end up doing the work in this issue. Let's pick things up over in https://github.com/phetsims/calculus-grapher/issues/290. Closing

pixelzoom commented 1 year ago

We didn't end up doing the work in this issue.

Oh I see. That's what "JUst kidding, we decided not to." must mean.