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

Important modifications to be applied to Workshop 2 #7

Closed
22 tasks done
pedrohbraga opened this issue Oct 21, 2020 · 5 comments · Fixed by #24
Closed
22 tasks done

Important modifications to be applied to Workshop 2 #7

pedrohbraga opened this issue Oct 21, 2020 · 5 comments · Fixed by #24
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@pedrohbraga
Copy link
Member

pedrohbraga commented Oct 21, 2020

Before developing and modifying this workshop, please read through the presenter and developer protocol carefully and refer back to it regularly as you work.

General issues to be addressed

Specific to adapting workshops to a remote framework:

  • Fit this workshop to a 3-hour format. The duration of this workshop should be ~3 hours (including exercises and one or two breaks of 5 minutes; excluding longer breaks). To fit this format, please add, edit, or hide slides from existing workshops. You can begin by hiding repetitive examples, non-essential slides, or some of the additional advanced slides that are included at the end of some workshops. Some workshops will be more difficult to shorten than others, due to their subject matter. It is not a problem to let these workshops run slightly over 3 hours to guarantee accurate and complete information. (Use your judgment). DO NOT delete slides. Use the instruction: hidden_remote tag. See an example below:
---
instruction: hidden_remote

# Slide title: I love statistics!

### Content: Amazing!
---
  • Develop new or adapt current exercises to better suit the remote workshop format and the use of breakout rooms. Exercises should be informative and engaging. Remember to leave time for students to work through exercises and ask questions. Refer to these guidelines to find suggestions of exercises promoting active learning. You may use additional resources to create the exercises.

Specific to slide structure, grammar and style

  • Ensure code is working correctly and that code matches workshop presentation. Continue to check this as you make changes.

  • Proofread correcting spelling, grammar, or punctuation errors.

  • Look for and correct the content of slides that overflow the screen.

  • Look for and correct prose that mentions function names, object names, package names and code chunks that do not use backticks.

  • Look for and correct unnecessary abbreviations (e.g. use do not instead of don’t).

  • Correct unnecessary long sentences.

  • Ensure that the slides are written in a consistent style, which helps with the flow of workshops and student understanding. This is especially important when working collaboratively with other developers.

  • Include presenter notes to the slides you modify. You can write notes for yourself to read in the presenter mode (press the keyboard shortcut p). These notes are written under three question marks ???. See the example below:

---
The content of this workshop is essential for your graduate studies.

???

Note: _The presenter should sing this slide to the participants_.
---

Specific to the translation of this workshop

  • Add and translate the modified content to the French version of the workshop.

  • Proofread and correct spelling, grammar and punctuation errors.

  • Correct unnecessary long sentences.

Specific to this workshop

  • Solve issue Challenge: use you data (slide 52) #10
  • Solve issue loading tidyverse a the end of the workshop #5 .
  • Solve issue namespace ... or not!  #6 .
  • About the workshop structure: Labeling the tidy content (dplyr, tidyr, and maggritr) as something for "advanced users" makes these tools seem less accessible, when that is not necessarily the case. This also breaks the workshop structure's logic, and makes this section seem separate from the rest of the workshop. In an effort to make this section less intimidating, it would be better to change this wording to focus on making the distinction between base R and the tidyverse sections of the workshop. Specifically, it can be important to briefly explain the contributions of the tidy packages (i.e. why use dplyr when most operations can be done in base R). This would help clarify the workshop structure, and would also be a great opportunity to make a fun "Welcome to the tidyverse" slide to introduce the tidy content 🤩.
  • Convert all gather() and spread() occurrences to pivot_wider() and pivot_longer(), because gather() and spread() are now retired. Argument names will also need to be changed (key and value have been replaced).
  • Update logos (see new dplyr logo here, tidyr logo here).
  • Slide 39: Change header = T to header = TRUE, because using T is not recommended.
  • This workshop should (briefly) introduce the tibble structure in the tidyverse section.
  • Optional inspiration for a challenge: asking participants to do the same manipulation and summary operations using base R and their tidy equivalents, perhaps as a final challenge in groups. This could be a great way to show how the tidy approach can yield shorter code.

Only close this issue after all cases have been ticked and verified.

@pedrohbraga pedrohbraga added bug Something isn't working enhancement New feature or request labels Oct 21, 2020
@pedrohbraga pedrohbraga pinned this issue Oct 21, 2020
@pedrohbraga pedrohbraga changed the title Important modifications to be applied to this workshop Important modifications to be applied to Workshop 2 Oct 21, 2020
@linley-sherin
Copy link
Contributor

linley-sherin commented Feb 10, 2021

Additional issues to be addressed (added Feb 2021)

Both presentations

  • Badges on slide 2 are out of data (need to be updated with the badges from the README)

English presentation

  • Slides 73, 84, 85 and 112 have content overflow
  • Slide 73 shows some md notation/part of the code that should be invisible
  • Slide 14 has a spelling error

French presentation

  • Slides 83, 84, 87, 89, 94 have content overflow
  • Many slides do not match the English presentation (ex. slides 6, 9, 10, 11,14, 20, 21, 23, etc). EDIT: Sometimes the English slides look better, sometimes the French. Use your discretion to modify the workshops so that they match and look as good as possible.

@VLucet
Copy link
Contributor

VLucet commented Aug 27, 2021

I have updated the badges in the french presentation and will do the same in the english pres on my branch otherwise decated to improving the french presentation val-french-pres. Please take note @YeonseonJeon

@VLucet
Copy link
Contributor

VLucet commented Aug 27, 2021

Cant find the overflow in slides 83, 84, 87, 89. Slide 94 has a small overflow o the roght which I fixed. @linley-sherin can you explain what you mena by overflow on slides 83, 84, 87, 89? I dont see it.
In addition, Im sorry but I dont see how the french prez has less text on slides 6, 9, 10 etc.. which makes me wonder if maybe this other action item is outdated? Can you confirm?

@linley-sherin
Copy link
Contributor

linley-sherin commented Aug 27, 2021

Cant find the overflow in slides 83, 84, 87, 89. Slide 94 has a small overflow o the roght which I fixed. @linley-sherin can you explain what you mena by overflow on slides 83, 84, 87, 89? I dont see it.
In addition, Im sorry but I dont see how the french prez has less text on slides 6, 9, 10 etc.. which makes me wonder if maybe this other action item is outdated? Can you confirm?

Hi Valentin! I can't find the overflow now either, I checked previous commits and don't see where it was fixed. I am not sure what happened there but I will run through the issues listed above right now and edit so that the list is accurate. Thank you for checking in.
edit: I think Pedro must have fixed it in the Winter because he was doing other edits at that time. Sorry for the confusion!

@linley-sherin
Copy link
Contributor

The issue list is updated now. I added a note that you can modify either the French or English slides to match each other because I noticed when I was looking through that sometimes the French slides had more text which you might want to keep. You can use your discretion in those instances. Feel free to check in with us at any point if you two are confused or need a 3rd opinion. Thanks!

@VLucet VLucet linked a pull request Sep 23, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants