-
Notifications
You must be signed in to change notification settings - Fork 6
Add live surveys to DS pathway #2245
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
Comments
…ection and make them ifproglang specific to address starter file warning (see #2245)
@retabak If you have a chance before our meeting on Tuesday, checkout the latest draft of the live survey instructions in |
@retabak please review this file: |
@schanzer We had a thought about launching the MOC lesson with a live survey. Possible to have the reactor generate mean median and mode? OR does this not make sense? |
@flannery-denny yeah, that's doable -- the
|
@schanzer Turns out bar and pie charts are in the same lesson with frequency tables, so our live survey needs to display them all together. I need a method to put the table beside the image here:
When you get a chance, please either send me a fix or update the file. Thanks! https://code.pyret.org/editor#share=1ZlKmA4Dx9SWXnk7hQx8FQ1AKHmykcsoE&v=fee2ecd |
@schanzer There's a note here about possibly using a live survey in linear 3. I'm not seeing the connection anymore... and am inclined to remove the bullet point unless you have a concrete idea (but I don't actually think the mention stems from you.) Other than that, will definitely be able to close this issue by Monday. Just need to fix the bar and pie chart starter file and add language to the MOC lesson. |
@schanzer once you fix (or let me know how to fix) the live survey starter file for bar and pie charts, this issue is ready to close and we can merge |
@flannery-denny I did tell you this was impossible earlier. ;) It turns out there is a hack - forcing the reactor to render its internal state, which gives us zero control over things like font size, cropping, image size, etc. This is the end result: ![]() I think this is a non-starter |
@schanzer agreed. I'll adjust the starter file and merge the pull request. |
@schanzer A new interesting noticing about live pyret surveys... the bar charts don't work like other bar charts in that they don't leave empty columns for the options in the google form that don't get populated in the table... any way to get this bar chart to include black, grey, green, other? It's a big point we make when comparing the two kinds of visualizations. ![]() |
Incorporate live surveys into DS lessons where we introduce new displays. Addresses #2245
@schanzer - opt-blocks are now rendering in slides, but there is no indicator on the slide that they are optional... I suggest we make a new Optional slide for the slide template that authors can specify in the slidebreak |
@flannery-denny how do you want to show teachers (and students?) that a slide is optional? Is it the background color? The icon on the template? |
@schanzer I suggest we make a new Optional slide for the slide template that authors can specify in the slidebreak. An optional icon could do the trick, but I think it actually has to have the word Optional in it. I hadn't thought of a different color background - it might be more obvious to teachers that there was something different. Perhaps the template should both have an icon with the word optional and a slightly different background color? |
These should be offered as an optional do now as the first time students see each new display for teachers who feel confident
shared/langs/en-us/docroot/pages/live-pyret-survey-instructions.adoc
Update the questions on these google form surveys
computing needs all voices - ketchup survey?not possible because there are blank cells in the table.~Update starter files and confirm that they 1) are commented so teachers can make sense of the code 2) do what we want them to do when we click "Run"
Incorporate the surveys into the lesson plans!
alg2 - linear 3I think I thought that it might make sense to reuse thelr-plot
starter file in this lesson, but it seems like a stretch to work it in at the moment.Other ideas we had:
The text was updated successfully, but these errors were encountered: