-
Notifications
You must be signed in to change notification settings - Fork 0
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
[CODAP] Wrapper should not reveal mystery launcher mechanism and angle stability #348
Comments
Note: This feature was requested and has been approved by HL's group. I really like the fact that it lets students 'solve' for the properties of the mystery launchers without needing to hide the variables for 'Launch mechanism' and 'Angle stability' because they would be spoilers. @catherinecarter are you okay with this change? If so, I think this issue can be closed. |
I think it's ok. I also thought giving that information away might be too much. I do, however, think that the documentation for how to include it as a query parameter needs to be accessible for users (specifically teachers). Can we include this in the dialog box that pops up when the sim opens, towards the bottom? Students are less likely to read the whole thing. Not many teachers (even power users) know about (and use) query parameters, so adding this to be more visible seems like a good idea. |
Thanks @catherinecarter, continuing that discussion in #350. This issue can be closed. |
Looks good in rc.1. I see "?" for the launcher mechanism and angle stability with mystery launchers. |
One of the exemplar lessons that we have designed for the PDL + CODAP wrapper has students investigating the mechanisms and angle stabilities of mystery launchers. Allowing them to directly access the 'Launch mechanism' and 'Angle stability' of the mystery launchers can undermine these lessons. It makes sense to include these values in the data stream for the custom launchers, but for the mystery launchers it would be better to give a value of '?' for these two quantities, since those are the 'mystery' of the mystery launchers.
The text was updated successfully, but these errors were encountered: