Skip to content

Fix IO warning#296

Merged
Zethson merged 10 commits intomainfrom
fix/warning
Jul 22, 2025
Merged

Fix IO warning#296
Zethson merged 10 commits intomainfrom
fix/warning

Conversation

@Zethson
Copy link
Copy Markdown
Member

@Zethson Zethson commented Mar 25, 2025

Fixes

/home/lukas/miniforge3/envs/lamindb/lib/python3.12/site-packages/anndata/utils.py:429: FutureWarning: Importing read_text from `anndata` is deprecated. Import anndata.io.read_text instead.
  warnings.warn(msg, FutureWarning)

also modernizes the setup a tiny but and removes ancient flake8 and other stuff

Zethson and others added 8 commits March 25, 2025 12:32
Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
@Zethson
Copy link
Copy Markdown
Member Author

Zethson commented Mar 25, 2025

IDK what to do about

Error: no downloadable artifacts found (expired)

@Zethson Zethson requested a review from LucaMarconato March 25, 2025 12:07
@Zethson
Copy link
Copy Markdown
Member Author

Zethson commented May 9, 2025

@LucaMarconato interested in merging this?

The RTD error is unrelated and it'll likely work with a rerun.

@Zethson
Copy link
Copy Markdown
Member Author

Zethson commented Jul 22, 2025

It's been 3 months and several requested reviews. I've checked the PR again and it should not change behavior. Therefore, I'll merge it now and am ready to take any blame if I broke something.

@Zethson Zethson merged commit 4ee33da into main Jul 22, 2025
3 of 4 checks passed
@LucaMarconato LucaMarconato deleted the fix/warning branch November 10, 2025 01:01
@LucaMarconato
Copy link
Copy Markdown
Member

Thanks!

IDK what to do about
Error: no downloadable artifacts found (expired)

To fix one had to re-run this https://github.com/scverse/spatialdata-io/actions/workflows/prepare_test_data.yaml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants