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

Delete moreWires and increase number of wires to 50 #884

Closed
arouinfar opened this issue May 31, 2022 · 3 comments
Closed

Delete moreWires and increase number of wires to 50 #884

arouinfar opened this issue May 31, 2022 · 3 comments

Comments

@arouinfar
Copy link
Contributor

@DianaTavares reported a user issue with the number of wires available by default.

Hi PhET design team! PhET fellows are taking more PhET simulations to their classes and today they report me some issues:
there is not more wire to build more circuits:
image

The recommended solution was to use the moreWires query parameter. However, @DianaTavares raised these concerns:

I start to think that the query parameter can be a problem in this way teachers are using the sims on cellphones. what Amanda said, if they are using the app, they can not put the qps. and if the teacher is sending the file (no the link, because WhatsApp is free in Mexico in most of the cellphone companies), students have to write very specific things in the URL of the file (text that doesn't have a sense for them) sound a lot of mistakes.

Query parameters are not accessible in the app and are hard to share when sending the .html file. There are two options we can consider:

  1. Add a "More Wires" checkbox to the Options dialog. (This would likely require changing the query parameter from a flag to a boolean.)
  2. Deprecate the query parameter and always include the full 50 wires in the simulation. We originally limited the number of wires due to performance concerns. However, when introducing moreWires in More wires? #432 the performance was acceptable on iPad and Chromebook.

I think we could go with option (2), but we should discuss as a team, so adding it to the PhET-iO publication milestone.

@DianaTavares
Copy link

option 2) solve the problem of this issue of more wires, but I think that moving from query parameter to something in the Options dialog can be better In general for all the simulations. I visualize that this can be a recurrent problem with other simulations, like the new math sims that have a lot of query parameters, or Natural selection where starting with a specific bunny population can be useful for some experiments.

@arouinfar arouinfar mentioned this issue Jun 2, 2022
5 tasks
@arouinfar
Copy link
Contributor Author

Discussed with @samreid and @kathy-phet and we decided to go with option (2) and delete the moreWires query parameter and include 50 wires by default.

@arouinfar arouinfar changed the title Make moreWires more accessible Delete moreWires and increase number of wires to 50 Jun 2, 2022
@samreid
Copy link
Member

samreid commented Aug 27, 2022

Fixed. Clients that continue to use ?moreWires from their bookmarks will not run into problems (query parameter will be safely ignored). Closing.

@samreid samreid closed this as completed Aug 27, 2022
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