Skip to content
New issue

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

Some values can't be customized using their associated model property #351

Closed
Tracked by #973
Nancy-Salpepi opened this issue Aug 15, 2023 · 6 comments
Closed
Tracked by #973

Comments

@Nancy-Salpepi
Copy link

Nancy-Salpepi commented Aug 15, 2023

For phetsims/qa#970

In the examples.md, under Control Sunlight and Infrared Properties, it says that values can be customized using the associated model property, but
greenhouseEffect.layerModelScreen.model.groundLayer.albedoProperty and
greenhouseEffect.layerModelScreen.model.atmosphereLayers.numberOfActiveAtmosphereLayersProperty
are read only and can only be customized through direct interaction with the sim.
Is that OK?

@arouinfar
Copy link
Contributor

Thanks for reporting @Nancy-Salpepi. These properties appear on all screens, which is probably why they are read-only. It would be problematic for clients to change the albedoProperty of the ground on the Waves screen, for example.

@jbphet can you make these properties phetioReadOnly: false on the Layer Model screen only? These properties power UI controls on that screen, and it's reasonable to expect that clients may need to set them with the API.

@jbphet
Copy link
Contributor

jbphet commented Aug 18, 2023

@arouinfar - I've committed changes for this, please check it over and assign back to me with your assessment.

@jbphet jbphet assigned arouinfar and unassigned jbphet Aug 18, 2023
@arouinfar
Copy link
Contributor

Things look great on main @jbphet. As requested, back to you.

@arouinfar arouinfar assigned jbphet and unassigned arouinfar Aug 21, 2023
@jbphet
Copy link
Contributor

jbphet commented Aug 21, 2023

Cool. I'll unassign and have it verified by QA in the next round of testing.

@jbphet jbphet removed their assignment Aug 21, 2023
@jbphet jbphet added status:fixed-awaiting-deploy dev:enhancement New feature or request status:ready-for-review and removed type:question Further information is requested labels Aug 21, 2023
@jbphet
Copy link
Contributor

jbphet commented Aug 22, 2023

QA - Feel free to close once verified.

@Nancy-Salpepi
Copy link
Author

The properties are now phetioReadOnly: false and customizable 🙂.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants