Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

adjusted docstrings in steinbock reader py file #165

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

LukasHats
Copy link

As discussed in here, I made a few comments in the docstrings of the steinbock reader to improve documentation and possibly also improvements to the code.

Also tagging @melonora here as we will sit together and have a look at how to improve flexibility or at least the documentation.

@melonora melonora marked this pull request as ready for review February 24, 2025 08:33
@melonora melonora closed this Feb 24, 2025
@melonora melonora reopened this Feb 24, 2025
@codecov-commenter
Copy link

codecov-commenter commented Feb 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.50%. Comparing base (54da345) to head (7811298).
Report is 382 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #165       +/-   ##
===========================================
+ Coverage   35.91%   49.50%   +13.58%     
===========================================
  Files          19       26        +7     
  Lines        1715     2705      +990     
===========================================
+ Hits          616     1339      +723     
- Misses       1099     1366      +267     
Files with missing lines Coverage Δ
src/spatialdata_io/readers/steinbock.py 38.77% <ø> (-3.45%) ⬇️

... and 21 files with indirect coverage changes

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants