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 sept oct nov workshops #474

Merged
merged 12 commits into from Dec 6, 2021
Merged

add sept oct nov workshops #474

merged 12 commits into from Dec 6, 2021

Conversation

raynamharris
Copy link
Collaborator

@raynamharris raynamharris commented Oct 18, 2021

add Intro to Conda, Intro to R, and RNA-Seq in the Cloud workshops offered in September and October

PR Checklist

PR Description

Briefly describe the changes this PR will add/address
Preview link
https://cfde-training-and-engagement--474.com.readthedocs.build/en/474/

Review format

Add how you would like to receive the review (direct editing, as comments, notes etc)

Timeline

Add expected timeline to finish review and merge adding note for time critical updates

@s-canchi
Copy link
Contributor

I don't see the added workshops in the built website. Hm..

@s-canchi s-canchi added Training-Release Label for PRs to be included in release notes fixes Label for PR merges related to fixes labels Nov 19, 2021
@raynamharris
Copy link
Collaborator Author

I added a few more workshops from November and fixed the October workshop links

@s-canchi
Copy link
Contributor

I still don't see the workshops in the rendered website (link in the top comment). Do you ? Could you try making the changes locally and pushing it to see if that works ?

@raynamharris
Copy link
Collaborator Author

No, I did not make changes locally. I would prefer to not have this repository cloned on my local computer if possible.

@s-canchi
Copy link
Contributor

s-canchi commented Dec 1, 2021

I am not sure why but for some reason the changes you made on GitHub don't seem to show up on the rendered website. I can make a new branch with the same additions and push it to check it that fixes it. Thus far, except minor text edits, we usually end up making changes locally before pushing it to this repo. That is mostly because we can see a local version of the rendered site prior to pushing changes.

@raynamharris raynamharris changed the title add sept and oct workshops add sept oct nov workshops Dec 3, 2021
@raynamharris
Copy link
Collaborator Author

This is finally on the built website :) Good to go.

Copy link
Contributor

@s-canchi s-canchi left a comment

Choose a reason for hiding this comment

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

LGTM!

@s-canchi s-canchi merged commit d082f68 into dev Dec 6, 2021
@s-canchi s-canchi deleted the raynamharris-patch-1 branch December 7, 2021 22:41
s-canchi added a commit that referenced this pull request Dec 10, 2021
* Update rna_seq_2.md

Made some small changes, adjusted wording to be more clear.

* Update rna_seq_3.md

* Update rna_seq_4.md

* Update rna_seq_5.md

* Update rna_seq_6.md

* Update rna_seq_7.md

* Update rna_seq_8.md

* Update rna_seq_1.md

* minor text edits for conda lesson

* Update rna_seq_2.md

* Update rna_seq_3.md

* Update rna_seq_3.md

* Update rna_seq_4.md

* Update rna_seq_4.md

* Adding workshop resources  (#469)

* adding conda workshop screenshots for attendees

* communication image

* resize communication image

* adding launch binder and export binder file imgs

* aws screen img

Co-authored-by: marisalim <marisa_lim@pacbell.net>
Co-authored-by: Marisa Lim <marisalim@users.noreply.github.com>

* RNAseq on Cavatica lesson fixes (#464)

* fixed image links across all lessons

* Added network access admonition box

* fixed error in number of files in cavatica-view lesson

* updated to RNA-Seq naming convention

* updated home page RNA-Seq carousal

* updated home pg carousal with conda lesson blurb

* added trainling mailing list to the workshops box on home pg

* added training mailing list to announce bar

* added july workshops to box on home pg

* updated lfc shrink text

* added mailing list icon in footer

* fixed broken links to kf rnaseq workflow

* correction to GTF reference file selection

* updated R scripts with GTF file

* updated deseq2 app lesson with new analysis setup vidlet and steps

* updated the vidlet to tag and download DESeq2 outputs

* fixed broken link

* correction to conda description in home pg

* corrections to data cruncher scripts

* updated KF lesson with RNA-Seq syntax

* updated intro to R workshop link on home pg

* updated lfc shrinkage text

* fixed right render issue on chrome

* updated workshop box with august wrkshps

* Lesson edits and minor fixes before merge to stable (#471)

* text corrections based on jeremy's suggestion

* standardize to RNA-Seq on landing pgs

* a->an and an oxford comma just because

Co-authored-by: marisalim <marisa_lim@pacbell.net>
Co-authored-by: Jeremy Walter <40613731+jeremywalter@users.noreply.github.com>

* clarify vidlet topics, add export option

per suggestions from Dirk's JOSE review

* add note about binder (base)

* updated reviewer list for PRs (#479)

* Updates to AWS tutorial  (#478)

* Updates to AWS tutorial with new screenshots and correction to AMI choice

* Fixed links and corrections to build statements

* Fixed links to open in new tab

* add sept oct nov workshops (#474)

* add sept and oct workshops

* add november workshops

* fix conda workshop link

* hyperlink calendar text not past workshop text

* add sep oct nov workshops

* consistent titles

* added august workshops and minor edits

* updated links and minor text fixes

* update comment pr workflow

Co-authored-by: Saranya Canchi <36385305+s-canchi@users.noreply.github.com>

* Update README.md (#480)

* Update README.md

* updated with august and july workshops

* added training mailing list

* fixed email link

* minor text edit

* updated email alias

Co-authored-by: Saranya Canchi <36385305+s-canchi@users.noreply.github.com>

* added release notes for december (#482)

Co-authored-by: HigginsD3b <63019615+HigginsD3b@users.noreply.github.com>
Co-authored-by: marisalim <marisa_lim@pacbell.net>
Co-authored-by: Marisa Lim <marisalim@users.noreply.github.com>
Co-authored-by: Jeremy Walter <40613731+jeremywalter@users.noreply.github.com>
Co-authored-by: Rayna M Harris <raynamharris@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixes Label for PR merges related to fixes Training-Release Label for PRs to be included in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants