Skip to content

fix: attempts to update geneformer notebooks#745

Merged
jomitchellnv merged 2 commits into
mainfrom
jm/geneformer-move-cellxgene-tutorial
Mar 22, 2025
Merged

fix: attempts to update geneformer notebooks#745
jomitchellnv merged 2 commits into
mainfrom
jm/geneformer-move-cellxgene-tutorial

Conversation

@jomitchellnv
Copy link
Copy Markdown
Collaborator

Description

Type of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactor
  • Documentation update
  • Other (please describe):

CI Pipeline Configuration

Configure CI behavior by applying the relevant labels:

Note

By default, the notebooks validation tests are skipped unless explicitly enabled.

Usage

TODO: Add code snippet

Pre-submit Checklist

  • I have tested these changes locally
  • I have updated the documentation accordingly
  • I have added/updated tests as needed
  • All existing tests pass successfully

Copy link
Copy Markdown
Collaborator

@skothenhill-nv skothenhill-nv left a comment

Choose a reason for hiding this comment

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

looks good

@jomitchellnv jomitchellnv enabled auto-merge March 14, 2025 02:14
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Mar 19, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@jomitchellnv jomitchellnv disabled auto-merge March 19, 2025 18:23
@jomitchellnv jomitchellnv force-pushed the jm/geneformer-move-cellxgene-tutorial branch 2 times, most recently from 28e225a to 7ff9d18 Compare March 19, 2025 18:33
Signed-off-by: Jonathan Mitchell <jomitchell@nvidia.com>
@jomitchellnv jomitchellnv force-pushed the jm/geneformer-move-cellxgene-tutorial branch from 7ff9d18 to b729044 Compare March 19, 2025 18:35
@jomitchellnv jomitchellnv enabled auto-merge March 19, 2025 19:10
@pstjohn
Copy link
Copy Markdown
Collaborator

pstjohn commented Mar 21, 2025

/ok to test

@jomitchellnv jomitchellnv force-pushed the jm/geneformer-move-cellxgene-tutorial branch from 563e1d3 to 9029efc Compare March 21, 2025 23:15
@jomitchellnv jomitchellnv disabled auto-merge March 21, 2025 23:24
@jomitchellnv jomitchellnv enabled auto-merge March 21, 2025 23:24
@pstjohn
Copy link
Copy Markdown
Collaborator

pstjohn commented Mar 22, 2025

/ok to test

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.15%. Comparing base (82c4b13) to head (9029efc).
⚠️ Report is 479 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #745   +/-   ##
=======================================
  Coverage   84.15%   84.15%           
=======================================
  Files         136      136           
  Lines        8529     8529           
=======================================
  Hits         7178     7178           
  Misses       1351     1351           

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jomitchellnv jomitchellnv added this pull request to the merge queue Mar 22, 2025
Merged via the queue into main with commit 32be470 Mar 22, 2025
@jomitchellnv jomitchellnv deleted the jm/geneformer-move-cellxgene-tutorial branch March 22, 2025 02:40
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.

5 participants