Skip to content

Conversation

ghaith-mq
Copy link

This pull request was created during the TheisLab Hackathon 2024 as part of a project to develop a cloud-based Spatial-DB. The primary goal is to ensure that when users upload spatial datasets to the database, they meet predefined quality and consistency standards.

To achieve this, we’ve introduced new functionality that:

  • Estimates pixel size (in micrometers) from Visium spatial transcriptomics data.

  • Extracts the image shape of the full-resolution tissue image.

These enhancements are critical for maintaining consistency when integrating spatial datasets from different sources, ensuring that users upload the correct full-resolution image and that spatial data is properly scaled for downstream analysis.
Inspired by Hest1k codebase

@ghaith-mq ghaith-mq marked this pull request as draft October 10, 2024 14:24
@codecov-commenter
Copy link

codecov-commenter commented Oct 10, 2024

Codecov Report

❌ Patch coverage is 16.36364% with 46 lines in your changes missing coverage. Please review.
✅ Project coverage is 44.75%. Comparing base (cff4896) to head (450bcd2).
⚠️ Report is 237 commits behind head on main.

Files with missing lines Patch % Lines
src/spatialdata_io/readers/visium.py 16.36% 46 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #219      +/-   ##
==========================================
- Coverage   45.50%   44.75%   -0.76%     
==========================================
  Files          22       22              
  Lines        2149     2203      +54     
==========================================
+ Hits          978      986       +8     
- Misses       1171     1217      +46     
Files with missing lines Coverage Δ
src/spatialdata_io/readers/visium.py 18.18% <16.36%> (-1.50%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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