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

GHA: Automated live (rendered) versions of the notebooks #821

Closed
wants to merge 1 commit into from

Conversation

alexslemonade-docs-bot
Copy link
Collaborator

Description:

This PR auto-generated from github actions running make-live.R with:

  • The latest notebooks on the master branch
  • The latest docker image on Dockerhub

Instruction for reviewers:

Make sure that all the notebooks have been made into _live versions appropriately and have the updates you expect.

@alexslemonade-docs-bot alexslemonade-docs-bot added the automated This PR was made automatically by github actions label Dec 4, 2024
Comment on lines +114 to +115
# ensure barcodes are set as column names in the SCE object

Copy link
Member

Choose a reason for hiding this comment

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

@jashapiro would you prefer this comment not be there (and I move it to the end of the line so it goes away) for when you teach this? I'm going to fix those spots I introduced in the integration notebook; let me know if you want me to fix this one too.

Copy link
Member

Choose a reason for hiding this comment

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

yeah, probably better to have it at the end of the line

Copy link
Member

Choose a reason for hiding this comment

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

👍


Once we have the rankings, we can calculate the AUC scores for both gene sets in all cells with the `AUCell_calcAUC()` function.

```{r calc_auc, live=TRUE}
Copy link
Member

Choose a reason for hiding this comment

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

add space here.

@sjspielman
Copy link
Member

Flagged items fixed in #823

@sjspielman sjspielman closed this Dec 4, 2024
@jashapiro
Copy link
Member

Flagged items fixed in #823

For the record, I don't think you had to close this PR; rerunning the action should have just updated the branch.

@sjspielman
Copy link
Member

I wasn't sure...so I figured this was safest. But noted 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated This PR was made automatically by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants