Skip to content

Add preliminary polygon query#248

Merged
kevinyamauchi merged 2 commits into
mainfrom
fixes/paper
May 10, 2023
Merged

Add preliminary polygon query#248
kevinyamauchi merged 2 commits into
mainfrom
fixes/paper

Conversation

@LucaMarconato
Copy link
Copy Markdown
Member

@LucaMarconato LucaMarconato commented May 10, 2023

This PR adds the functionality to query SpatialData objects shapes elements.

@LucaMarconato LucaMarconato requested review from giovp and kevinyamauchi and removed request for giovp May 10, 2023 00:30
@codecov
Copy link
Copy Markdown

codecov Bot commented May 10, 2023

Codecov Report

Merging #248 (15d6169) into main (5838653) will decrease coverage by 1.44%.
The diff coverage is 12.04%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #248      +/-   ##
==========================================
- Coverage   86.13%   84.69%   -1.44%     
==========================================
  Files          35       35              
  Lines        4225     4307      +82     
==========================================
+ Hits         3639     3648       +9     
- Misses        586      659      +73     
Impacted Files Coverage Δ
src/spatialdata/models/__init__.py 100.00% <ø> (ø)
src/spatialdata/_core/query/spatial_query.py 75.80% <6.45%> (-17.34%) ⬇️
src/spatialdata/models/_utils.py 76.92% <25.00%> (-14.63%) ⬇️
src/spatialdata/__init__.py 100.00% <100.00%> (ø)

Copy link
Copy Markdown
Collaborator

@kevinyamauchi kevinyamauchi left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you @LucaMarconato

@kevinyamauchi kevinyamauchi changed the title Fixes/paper Add preliminary polygon query May 10, 2023
@kevinyamauchi kevinyamauchi merged commit c464772 into main May 10, 2023
@kevinyamauchi kevinyamauchi deleted the fixes/paper branch May 10, 2023 09:41
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