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

Add psychological safety slides as a Quarto thing #41

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mfisher87
Copy link
Member

@mfisher87 mfisher87 commented May 12, 2024

Resolves #40

@mfisher87 mfisher87 force-pushed the psychological-safety-quarto-slides branch from c345519 to d3c9fb1 Compare May 12, 2024 18:06
@mfisher87 mfisher87 force-pushed the psychological-safety-quarto-slides branch from d3c9fb1 to 79f655f Compare May 12, 2024 18:06
Co-Authored-By: Tara Robertson <>
Co-Authored-By: Julia Stewart Lowndes <[email protected]>
Co-Authored-By: Liz Neeley <[email protected]>
Co-Authored-By: Alexis Hunzinger <[email protected]>
Co-Authored-By: Andy Teucher <[email protected]>
Co-Authored-By: Luis López <[email protected]>
Co-Authored-By: Yuvi Panda <[email protected]>
@mfisher87
Copy link
Member Author

I attempted to mention everyone that helped review this and work on it in previous years as co-authors, but as it is with any attempt to credit everyone, I've surely forgotten many. Let me know who I need to add (and their e-mail address) and I'll fix it up!

@mfisher87 mfisher87 marked this pull request as ready for review May 12, 2024 19:03
@jules32
Copy link
Contributor

jules32 commented May 15, 2024

Thank you Matt! This looks great to me and thank you for also adding the .bib file! I have not built locally to have a look before merging; @stefaniebutland would you please review/render and then merge? Thank you!

@jules32 jules32 requested a review from stefaniebutland May 15, 2024 20:41
Copy link
Member

@stefaniebutland stefaniebutland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mfisher87 Thanks so much for thinking through and taking this extra step to contribute your quarto slides. I have pulled your branch locally and rendered it; for some comments, I'm not sure if what I'm seeing is due to that, or whether they do indeed need fixing.

  • Would you please restore the link to Tara's slides and citing her and add yours, rather than replacing? Many (most?) of us will continue to use Google Slides and some will also want to use that deck
  • Given this is first instance of sharing a quarto deck here, it warrants a "this presentation was made with quarto ..." brief thing so folks know not to expect a google slide deck.
  • Would also be great if you could keep link to your recording of the presentation; valuable for future presenters
  • emoji's are showing as :text only
  • RevealJS version (screenshot)
    • emojis appear as text
    • appears as a single page rather than a slide deck; says "Advance" between slides
Screenshot 2024-05-16 at 4 31 14 PM

I made one further comment in README.md

I hope this is helpful. Let me know if there's something else I should be doing to render locally to view things as intended.

* [Install quarto](https://quarto.org/docs/get-started/)
* `quarto preview`

While this preview server is running, you can make changes to files and view them in the
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you clarify where I would run quarto preview to view changes in browser? I imagine in Terminal, so do I have to have cloned the repo locally first?

@mfisher87
Copy link
Member Author

mfisher87 commented May 17, 2024

Hey Stef, thanks for your review!

Would you please restore the link to Tara's slides and citing her and add yours, rather than replacing? Many (most?) of us will continue to use Google Slides and some will also want to use that deck

Not sure if you had seen this, but I moved that in to a dedicated acknowledgement slide. My thinking was having it in the slide would ensure that acknowledgement was coupled with the content. I'm happy to restore it in both locations as well.

Given this is first instance of sharing a quarto deck here, it warrants a "this presentation was made with quarto ..." brief thing so folks know not to expect a google slide deck.

Where are you thinking this should go? Maybe I could create a dedicated page with some introductory instructions to using Quarto (revealJS) slide decks?

Would also be great if you could keep link to your recording of the presentation; valuable for future presenters

👍

emoji's are showing as :text: only

The emojis issue was a setting I forgot (fixed in f61cffb). But there are other issues in your screenshot. How are you rendering? The text "Advance" in your screenshot is actually part of a speaker note and shouldn't be visible, for example.

Can you clarify where I would run quarto preview to view changes in browser? I imagine in Terminal, so do I have to have cloned the repo locally first?

Correct. I'll add this detail as well!

I'd like to spend some time to set up an in-GitHub workflow for this with devcontainers/codespaces, but need to find the time :)

Has everyone been interacting with this in RStudio so far? Or a mix of RStudio + Quarto users? I've never used RStudio so I'd like to accurately capture any needed info in the README.

If you're using Quarto can you share output of quarto check?

@mfisher87
Copy link
Member Author

mfisher87 commented May 17, 2024

A thought came to me; in the slides we use the word "punish", but don't really define it. I think it's important to clarify that someone is "punished", in the learning sense, any time they experience negative feedback from their environment (which includes people). Punishment can look like feeling guilty for perceiving yourself as interrupting someone or wasting their time, even if that person doesn't actually feel that way. Not sure how to work that in.

@stefaniebutland
Copy link
Member

Good idea to meet briefly for us to get a common understanding and discuss the ideas you shared above. Thank you for suggesting that.

Would you please restore the link to Tara's slides and citing her and add yours, rather than replacing? Many (most?) of us will continue to use Google Slides and some will also want to use that deck

Not sure if you had seen this, but I moved that in to a dedicated acknowledgement slide. My thinking was having it in the slide would ensure that acknowledgement was coupled with the content. I'm happy to restore it in both locations as well.

Yes please restore in both locations. Most folks who teach this in the near future (usually Julie) will use/ update Google slides so they need to be able to see those up front, and they may choose to incorporate some of your work into the Google Slide deck. We don't have plans now to change over to making slides in Quarto; that's not where most of our community is at and we often have guest teachers who do not use Quarto.

emoji's are showing as :text: only

The emojis issue was a setting I forgot (fixed in f61cffb). But there are other issues in your screenshot. How are you rendering? The text "Advance" in your screenshot is actually part of a speaker note and shouldn't be visible, for example.

Your slides now appear as slides! That fixed "Advance", emojis, etc

Has everyone been interacting with this in RStudio so far? Or a mix of RStudio + Quarto users? I've never used RStudio so I'd like to accurately capture any needed info in the README.

If you're using Quarto can you share output of quarto check?

I use RStudio and Julie primarily uses RStudio. It comes with Quarto

stefaniebutland$ quarto check

[✓] Checking versions of quarto binary dependencies...
      Pandoc version 3.1.1: OK
      Dart Sass version 1.55.0: OK
[✓] Checking versions of quarto dependencies......OK
[✓] Checking Quarto installation......OK
      Version: 1.3.450
      Path: /Applications/quarto/bin

[✓] Checking basic markdown render....OK

[✓] Checking Python 3 installation....OK
      Version: 3.11.4
      Path: /opt/homebrew/opt/[email protected]/bin/python3.11
      Jupyter: (None)

      Jupyter is not available in this Python installation.
      Install with python3 -m pip install jupyter

[✓] Checking R installation...........OK
      Version: 4.4.0
      Path: /Library/Frameworks/R.framework/Resources
      LibPaths:
        - /Library/Frameworks/R.framework/Versions/4.4-arm64/Resources/library
      knitr: 1.46
      rmarkdown: 2.26

[✓] Checking Knitr engine render......OK

@stefaniebutland
Copy link
Member

Holding till Matt has time for us to revisit

@mfisher87
Copy link
Member Author

Hey Stef, sorry for the delay getting back to this! I put a meeting on your calendar for Monday :)

@stefaniebutland
Copy link
Member

no apology needed! Thank you. see you Monday

@mfisher87
Copy link
Member Author

Notes from our chat today:

  • Context: This repository, especially the core portion, is official material and so that's why it's the most highly controlled. We want it to be accessible to its current audience, which expects slides to be in Google.
  • Decision: There's already a link in "Additional slides and recordings" but it points to the external repo, change that link to point to the copy that's being uploaded in this PR.
  • Consequences: Folks who use these materials will be able to continue to contribute and use the materials the way they expect to.

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

Successfully merging this pull request may close these issues.

Add Quarto slides for team culture / psych. safety lesson
3 participants