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 slides for scRNAseq intro & differential expression #15

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

jashapiro
Copy link
Member

The main goal here is to add two sets of slides as part of #4.

I made only slight formatting edits to the slides, so I would not expect any real trouble.

However, I was very annoyed by the very long lines everywhere in the table, so I cut off the trailing right | and the associated spaces. This should make future edits much more pleasant, without affecting the rendered table at all as the outer pipes are optional.

reformat table to remove long lines
@jashapiro jashapiro requested a review from sjspielman December 4, 2024 00:53
Copy link
Member

@sjspielman sjspielman left a comment

Choose a reason for hiding this comment

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

Slides look good, and schedule format looks good! Just two things I noticed:

  • Intro slides: I think it makes sense to mention briefly (one bullet or sub-bullet on the last slide) that quantitative approaches to evaluate cluster quality exist. Just a little teaser, no details!
  • DEG slides: Something rather wonky appears to have happened during the PDF conversion on slides 9-10. I just tried re-exporting, and indeed google slides did the weird thing again. I suspect we just need to make these diagrams into separate images rather than making them with in google slides directly :/

@jashapiro
Copy link
Member Author

Thanks @sjspielman: I added a line about quantitative evaluation of clustering. I also recreated the PDF for the DEG slides by downloading a powerpoint version and saving that as PDF locally, which seemed to work fine.

@jashapiro jashapiro requested a review from sjspielman December 4, 2024 15:08
Copy link
Member

@sjspielman sjspielman left a comment

Choose a reason for hiding this comment

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

👍

| **Day 1** | **2024-12-10**
| 9:00 AM | Welcome, Introductions and Getting Started
| 10:00 AM | Introduction/Refresher to Single-cell Analysis with R and Bioconductor
| | [ Single-cell Analysis with R slides (PDF) ](../slides/2024-12-10_scRNAseq-data-intro.pdf)
Copy link
Member

Choose a reason for hiding this comment

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

did you mean to have spaces here actually? same Q for deg slides

Copy link
Member Author

Choose a reason for hiding this comment

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

Not necessarily, but they don't matter at all.

@jashapiro jashapiro merged commit 348a365 into main Dec 4, 2024
1 check passed
@jashapiro jashapiro deleted the jashapiro/intro-diff-slides branch December 4, 2024 15:12
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.

2 participants