We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
From #156, I am not certain how to test this. This sim is listed as a phet-io sim so it should probably be considered.
TANDEM: PhET-iO instrumented PhetioObject instances should be disposed.
@zepumph what should I be looking for for this?
The text was updated successfully, but these errors were encountered:
Oh, I just saw his documentation in implementation-notes.md
There are no dynamic pieces to this simulation, and as a result nothing is disposed. As a result no listeners need to be unlinked or removed.
So maybe this can just be closed. But perhaps there are things to think about still, so would be helpful for @zepumph to comment.
Sorry, something went wrong.
I ran fuzz/memory tests with this sim in studio and saw similar memory profiles to #139, so I don't see evidence of a leak.
I think of tandem references as similar to axon listeners in terms of managing disposal. I don't think there is anything left here.
jessegreenberg
No branches or pull requests
From #156, I am not certain how to test this. This sim is listed as a phet-io sim so it should probably be considered.
@zepumph what should I be looking for for this?
The text was updated successfully, but these errors were encountered: