Skip to content

Conversation

@d33bs
Copy link
Member

@d33bs d33bs commented Dec 1, 2025

Description

This PR adds a cloud-based CSV demonstration alongside the SQLite demonstration within the example notebook for cloud-based work. Along the journey towards this I also found there was a small bug with cloud-based SQLite file processing and fixed it (had to do with cached results).

What is the nature of your change?

  • Bug fix (fixes an issue).
  • Enhancement (adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).
  • This change requires a documentation update.

Checklist

Please ensure that all boxes are checked before indicating that a pull request is ready for review.

  • I have read the CONTRIBUTING.md guidelines.
  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • New and existing unit tests pass locally with my changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have deleted all non-relevant text in this pull request template.

@d33bs d33bs requested a review from gwaybio as a code owner December 1, 2025 22:15
@d33bs d33bs changed the title Csv cloud demo Cloud-based CSV example notebook additions Dec 1, 2025
@d33bs d33bs requested a review from jenna-tomkinson December 1, 2025 22:19
Copy link
Member

@jenna-tomkinson jenna-tomkinson left a comment

Choose a reason for hiding this comment

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

LGTM! I add some questions and comments but overall looks like a simple and clean addition to demonstrate CytoTables ability to work on cloud with CSVs :)

d33bs and others added 5 commits December 2, 2025 09:53
Co-Authored-By: Jenna Tomkinson <107513215+jenna-tomkinson@users.noreply.github.com>
Co-Authored-By: Jenna Tomkinson <107513215+jenna-tomkinson@users.noreply.github.com>
Co-Authored-By: Jenna Tomkinson <107513215+jenna-tomkinson@users.noreply.github.com>
@d33bs
Copy link
Member Author

d33bs commented Dec 2, 2025

Thanks @jenna-tomkinson !

@d33bs d33bs merged commit 0e875d3 into cytomining:main Dec 2, 2025
10 checks passed
@d33bs d33bs deleted the csv-cloud-demo branch December 2, 2025 17:39
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.

2 participants