-
-
Notifications
You must be signed in to change notification settings - Fork 33
Bringing feature/new-rstudio-server up-to-date with master
#938
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
Merged
Merged
Changes from all commits
Commits
Show all changes
199 commits
Select commit
Hold shift + click to select a range
13978b1
also link filtered for demo purposes
sjspielman 654d2e7
remove space so the link is a link
sjspielman 574c5a6
add alpha=1 to guide override
sjspielman c119d39
add a little sentence about patient diagnoses to help us remember tha…
sjspielman 398da1d
typos and formatting in gsea
allyhawkins f01b271
backticks for AUCell and SCE
allyhawkins 0e8e3d3
Merge pull request #868 from AlexsLemonade/allyhawkins/pathway-typos
allyhawkins c4b4082
Update scRNA-seq-advanced/02-dataset_integration.Rmd
sjspielman 7454d23
Merge branch 'master' into sjspielman/856-scadvanced-updates
sjspielman e0d21f5
Merge pull request #865 from AlexsLemonade/sjspielman/856-scadvanced-…
sjspielman ad068f5
reformat multi-line functions in intro
jashapiro d7bc902
Formatting for differential expression
jashapiro aae36f1
less live path typing
sjspielman 5384e6a
Merge pull request #870 from AlexsLemonade/jashapiro/sc-advanced-changes
jashapiro 4184775
Merge branch 'master' into sjspielman/863-dont-make-me-type
sjspielman 9311c72
remove non-detected genes
allyhawkins 1f4ec21
Merge pull request #872 from AlexsLemonade/allyhawkins/aucell-remove-…
allyhawkins 83d364c
Merge branch 'master' into sjspielman/863-dont-make-me-type
sjspielman 5d21965
rip out glue::glue
334f51c
updated merging preamble and sectioning, opened with checking genes
27fe546
sections for exploring coldata names and assays/reddims
2281386
rowdata section and function update
2c97955
Merge pull request #871 from AlexsLemonade/sjspielman/863-dont-make-m…
sjspielman 6dce13f
≈=Merge branch 'master' into sjspielman/integration-update
8547ac2
formatting updates
55569dd
update the technical merge diagram
sjspielman 41f3672
wording
sjspielman b043ffe
spelling
sjspielman fc09cc6
Apply suggestions from code review
sjspielman 3357a69
responded to merge section reviews
c88dc48
PCA and UMAP simplified
f309fa4
missed spots of merged_umap
84d89eb
Apply suggestions from code review
sjspielman cde10f3
Apply suggestions from code review
sjspielman 74c245b
Apply suggestions from code review
sjspielman 8813590
clean up deprecated comments, move up in function
f900ff0
parallel headers
sjspielman ab4ac11
one more
sjspielman 0ca6cfb
moar headers
sjspielman a7433ac
back up export
sjspielman 02c0f50
hands
sjspielman e48af16
Merge pull request #873 from AlexsLemonade/sjspielman/integration-update
sjspielman ae92db6
Live and rendered notebooks
sjspielman 5e71783
Merge pull request #875 from AlexsLemonade/auto_render_live
sjspielman 77d8ff9
2025-dec release
sjspielman 35e547b
Copy exercises from exercise-notebook-answers
sjspielman 852288d
Merge pull request #877 from AlexsLemonade/auto_copy_exercises
sjspielman da6e6ff
Merge branch 'master' into sjspielman/2025-dec-current-modules
sjspielman 7fa4261
Merge pull request #876 from AlexsLemonade/sjspielman/2025-dec-curren…
sjspielman 19d3dce
december not dec
sjspielman e0bd616
Merge pull request #878 from AlexsLemonade/sjspielman/dec-is-not-dece…
sjspielman 1c3d618
update syntax
sjspielman 47834f0
Update .github/workflows/build-docker.yml
sjspielman b1a3a77
Merge branch 'master' into sjspielman/879-jq-syntax
sjspielman 32c7657
Merge pull request #880 from AlexsLemonade/sjspielman/879-jq-syntax
sjspielman 7d7374e
remove old renv environment
sjspielman 2880ab9
bump renv
sjspielman bae48ff
restore the .Rprofile file now that we're bumped
sjspielman 3e8aaf4
verions to json
sjspielman 7db65b4
psm
sjspielman fbbcfa6
3.19 -> 3.22
sjspielman f5f4e94
remove the temp fix for Rsamtools, to see if it works now
sjspielman 36b0b51
Apply suggestions from code review
sjspielman ddc3090
Apply suggestions from code review
sjspielman 743a922
Merge pull request #884 from AlexsLemonade/sjspielman/869-bump-to-r4.…
sjspielman 3a2e4d9
bump renv
jashapiro 3ff1afb
Update snapshot
jashapiro e8da342
Merge pull request #889 from AlexsLemonade/jashapiro/renv117
jashapiro fd00552
Make lockfile consistent
jashapiro 786f6cc
Try to create a dual-platform build
jashapiro 3057a54
Merge pull request #890 from AlexsLemonade/jashapiro/multi-arch-docker
jashapiro e1676bb
remove platform flags now that we have multi-platform builds
jashapiro 6cea2a5
Merge pull request #891 from AlexsLemonade/jashapiro/no-platform
jashapiro 8b88b37
snapshot tximeta 1.28.3
sjspielman f31f72b
add scrapper to dependencies
sjspielman b04d684
snapshot scrapper and its dependencies
sjspielman e8d71d8
new line
sjspielman adc92e5
Merge pull request #905 from AlexsLemonade/sjspielman/895-899-tximeta…
sjspielman b8f67c3
scRNA-seq celltype notebook: update from restrict -> test.genes, incl…
sjspielman 4652281
some spacing while we're here
sjspielman 16c114b
update irlba record
sjspielman 603abb2
try adding libsqlite3-dev to docker install
sjspielman a8d7304
move to r section
sjspielman e4ef513
take it back out, seems like it's already there
sjspielman a59d30d
try for standalone install
sjspielman 77f71b9
revert a59d30d
sjspielman 3bb8287
renv update
sjspielman 3973df7
Apply suggestions from code review
sjspielman 6b74ab9
Apply suggestion from @sjspielman
sjspielman 3e9d40a
Merge pull request #908 from AlexsLemonade/sjspielman/bump-irlba
sjspielman 5ce695a
Merge branch 'master' into sjspielman/897-update-singler-bioc3.22
sjspielman e8952da
advanced GSEA notebook: spacing in general, and update the msigdbr ar…
sjspielman 0a6ee2b
fix typo and one more wording spot
sjspielman 686bcdd
advanced AUCell notebook: spacing all around, update AUCell to use a …
sjspielman 015fb8f
advanced aucell util: update plot_recovery_curve function to use list…
sjspielman 17fba61
fix typo
sjspielman 265a4d3
comment out unfixed notebooks with TODO linking #911
sjspielman a76a419
#902: update ORA notebook
sjspielman b1cce45
#903: update GSEA notebook
sjspielman 09328dc
#904: update GSVA notebook
sjspielman 1737eee
a couple more purrr spots in GSVA
sjspielman 2f913e7
temp comment out notebooks not to render, see if this one makes it th…
sjspielman 88e3471
might as well try same step we use in exercises repo
sjspielman c3deda2
Revert "might as well try same step we use in exercises repo"
sjspielman 2fafb08
Make sure that rm steps execute
jashapiro 46402f9
Merge pull request #913 from AlexsLemonade/jashapiro/shrink-renv
jashapiro 7ce97f7
Merge branch 'master' into sjspielman/897-update-singler-bioc3.22
sjspielman aaaf2ca
Merge branch 'master' into sjspielman/898-update-gsea-aucell
sjspielman bec676f
Merge pull request #906 from AlexsLemonade/sjspielman/897-update-sing…
sjspielman 7bdabe0
merge
sjspielman 9849d7a
Merge branch 'sjspielman/898-update-gsea-aucell' of github.com:AlexsL…
sjspielman 40cbf57
fix the bad merge
sjspielman b787158
remove extra period
sjspielman dcb48c5
update roadmap to close out #881 while I'm here
sjspielman ac35f60
Merge branch 'master' into sjspielman/902-903-904-updates
sjspielman b3733e3
all uncommented now
sjspielman 4ebbdd3
from #798, use dir_create and set a seed in gsea notebook
sjspielman 28ed7a3
woops this isnt stacked so need to comment out the other notebooks be…
sjspielman bbb1f29
separate coldata_joined variable to avoid bugs during instruction
sjspielman 4a24421
fix a couple spots where we have typos for sample ids
sjspielman 29bf271
get the other data frame naming in ORA
sjspielman fa3f254
update the salmon link since the note of interest is just above the l…
sjspielman 5026246
Merge pull request #910 from AlexsLemonade/sjspielman/898-update-gsea…
sjspielman 56daea9
all notebooks can run through ci now
sjspielman f220599
Merge branch 'master' into sjspielman/902-903-904-updates
sjspielman ae6159f
Clear space and render in docker container
jashapiro 5baa9a0
Update make-live workflow
jashapiro ef25540
Merge branch 'master' into sjspielman/901-and-other-rnaseq-updates
sjspielman 4bfdc26
update 01 tidyverse notebook to use built-in penguins
sjspielman 10cf1f5
remove palmerpenguins from renv
sjspielman 75218e2
spacing and use :: all around
sjspielman fa24aef
add sentence
sjspielman ba2499b
get rid of socviz link and add ggplot2 book
sjspielman 75829ed
add sentence to save to variable
sjspielman a6ecff6
Update pathway-analysis/02-gene_set_enrichment_analysis.Rmd
sjspielman b19f99b
Merge pull request #914 from AlexsLemonade/sjspielman/902-903-904-upd…
sjspielman bcc8f14
add core spatial packages and their deps to renv
sjspielman babf57f
Merge branch 'master' into jashapiro/gha-manual-docker
jashapiro bf8ab7e
use fixed path
jashapiro abf8cba
Try no user setting
jashapiro b0d9499
add free space reporting
jashapiro 528c202
add spatial directory and start off notebook in a project
sjspielman ded0ee1
Merge pull request #919 from AlexsLemonade/jashapiro/gha-manual-docker
jashapiro 3131b1d
draft import section with smidge of viz
sjspielman 2ee8700
Merge branch 'master' into sjspielman/917-initial-spatial-packages
sjspielman e121490
Merge branch 'master' into sjspielman/901-and-other-rnaseq-updates
sjspielman b79f8fc
restore sample id
sjspielman 6647e30
link the other samples fastqc
sjspielman 5ddb76f
Update scripts/link-data.sh
sjspielman 7b1c75e
update igraph - there was a new patch release
sjspielman bfc51ba
merge
sjspielman 543cf54
Merge pull request #915 from AlexsLemonade/sjspielman/901-and-other-r…
sjspielman a5236bc
run renv::update, which notably updates reticulate too which is now f…
sjspielman 9e789f0
Merge branch 'master' into sjspielman/917-initial-spatial-packages
sjspielman 8db0c0e
Merge pull request #926 from AlexsLemonade/sjspielman/917-initial-spa…
sjspielman 27210eb
Merge branch 'master' into sjspielman/920-draft-import
sjspielman 5e97285
ignore data
sjspielman d524083
fix a couple spots and stub out next sections
sjspielman 57e4245
add to render live script
sjspielman 5f6891e
typos
sjspielman 5ca0a5a
temporarily, test faster
sjspielman f1f48bb
revert temp script change as I flew too close to the sun
sjspielman dd574dd
Apply suggestions from code review
sjspielman afea9d3
dont save the h&e at this time
sjspielman c40c315
definitely mention no cytassist image since it affects import, and up…
sjspielman 0e51c2c
lil rephrasing now, of course much more to come
sjspielman 129d5a1
just load spatialexperiment, rest :: only
sjspielman c21d837
cytaimage out of dictionary
sjspielman 18d3777
slashes
sjspielman 8ff315e
Update spatial/01-spatial_intro.Rmd
sjspielman aa2b02f
pull out path defs being given to visiumio, and show sce-esque slots …
sjspielman 1959e5f
Update spatial/01-spatial_intro.Rmd
sjspielman 7ae5960
define sample id separately for typing
sjspielman 409b22e
micro
sjspielman e0edae5
Merge pull request #927 from AlexsLemonade/sjspielman/920-draft-import
sjspielman a50d201
Merge branch 'master' into sjspielman/860-909-updates
sjspielman ad0bdbd
Apply suggestion from @sjspielman
sjspielman d4af33f
add section to filter empty spots
sjspielman 5333548
add global filtering section
sjspielman 8818287
dont let figures get too big, we need to keep the html small
sjspielman f6008ba
a little bigger
sjspielman 35f0c3d
fix typo and add mito list to S3 in parallel
sjspielman c2055da
Use binaries for renv restore
jashapiro 7e20878
minor bugfix and formatting
jashapiro c323baa
Merge pull request #930 from AlexsLemonade/jashapiro/binary-docker
jashapiro b9dbe17
Merge branch 'master' into sjspielman/860-909-updates
jashapiro b041c46
Apply suggestions from code review
sjspielman a785e36
modify levels sentence since we actually cant see these 3 levels in t…
sjspielman ed254be
add num to dict
sjspielman 55061d3
Merge pull request #925 from AlexsLemonade/sjspielman/860-909-updates
sjspielman 314edb1
Merge branch 'master' into sjspielman/921-draft-filtering-part1
sjspielman 063b562
new fig sizing style
sjspielman 2b3f02b
respond to PR reviews
sjspielman 0c4d0ce
permeabilized is indeed a word
sjspielman 3361423
new line after #|
sjspielman 36ad2d5
patchwork:: and distinct colors for spot qc plots, avoiding orange fo…
sjspielman 1e9308e
border around H&E
sjspielman d8510f8
more ggplot theme more better
sjspielman 2445ee0
Merge pull request #929 from AlexsLemonade/sjspielman/921-draft-filte…
sjspielman File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,26 +1 @@ | ||
| source("renv/activate.R") | ||
|
|
||
| # Set the repos using the renv.lock file | ||
| renv_json <- jsonlite::read_json("renv.lock") | ||
| renv_r_repos <- renv_json$R$Repositories | ||
|
|
||
| # Extract the names | ||
| repo_names <- purrr::flatten_chr( | ||
| purrr::map(renv_r_repos, | ||
| ~ .x$Name) | ||
| ) | ||
|
|
||
| # Extract the URLs | ||
| repo_urls <- purrr::flatten_chr( | ||
| purrr::map(renv_r_repos, | ||
| ~ .x$URL) | ||
| ) | ||
|
|
||
| # Set the repo names | ||
| names(repo_urls) <- repo_names | ||
|
|
||
| # Set the options | ||
| options(repos = repo_urls) | ||
|
|
||
| # Remove all these objects | ||
| rm(renv_json, renv_r_repos, repo_names, repo_urls) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Copilot Autofix
AI about 2 months ago
In general, the fix is to add an explicit
permissions:block limitingGITHUB_TOKENto the least privileges required. Since this workflow only checks out the code, uses Docker and external actions, and uploads/downloads artifacts, it does not need to write to the repository. The minimal safe default ispermissions: contents: readat the workflow (root) level, which will apply to bothbuildandmergejobs.Concretely, in
.github/workflows/build-docker.yml, add a top-levelpermissions:block after thename:andon:keys (or directly afteron:/beforeenv:) withcontents: read. No jobs appear to require any other token scopes such aspull-requests,packages, orid-token. No imports or other code changes are needed; this is purely a YAML configuration adjustment.