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.

Sign up for GitHub

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

Jump to bottom

update tutorial #85

Closed
wants to merge 9 commits into from
Closed

update tutorial #85

wants to merge 9 commits into from

Conversation

zktuong
Copy link
Owner

@zktuong zktuong commented Oct 31, 2023

addresses #84

@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@codecov Codecov
Copy link

codecov bot commented Oct 31, 2023 •
edited
Loading

Codecov Report

Merging #85 (eaf44ad) into master (b70d98b) will decrease coverage by 0.57%.
The diff coverage is 52.21%.

❗ Current head eaf44ad differs from pull request most recent head a1544b6. Consider uploading reports for the commit a1544b6 to get more accurate results

@@            Coverage Diff             @@
##           master      #85      +/-   ##
==========================================
- Coverage   71.14%   70.58%   -0.57%     
==========================================
  Files           9        9              
  Lines        2010     2111     +101     
==========================================
+ Hits         1430     1490      +60     
- Misses        580      621      +41
Files Coverage Δ
R/utils.R 81.92% <100.00%> (+0.41%) ⬆️
R/plot_cpdb.R 64.96% <47.36%> (-2.50%) ⬇️

@zktuong zktuong self-assigned this Nov 1, 2023
zktuong
zktuong commented Nov 1, 2023
View reviewed changes
Copy link
Owner Author

@zktuong zktuong left a comment

Choose a reason for hiding this comment

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

looks ok

@zktuong zktuong enabled auto-merge (squash) November 1, 2023 00:01
@zktuong
Copy link
Owner Author

zktuong commented Nov 1, 2023

seems to introduce some unfixable error here

@zktuong zktuong closed this Nov 1, 2023
auto-merge was automatically disabled November 1, 2023 10:03

Pull request was closed

@zktuong zktuong deleted the update-tutorial-with-degs branch April 2, 2024 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@zktuong zktuong

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@zktuong