Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix xenium default value and add deprecation warning #186

Merged
merged 3 commits into from
Aug 1, 2024

Conversation

LucaMarconato
Copy link
Member

@LucaMarconato LucaMarconato commented Aug 1, 2024

This PR is AI generated (Copilot Workspace).

Related to #184

Revert the default value of cells_as_circles to True and add a deprecation warning.

  • Change the default value of the cells_as_circles parameter to None in the xenium function.
  • Add a deprecation warning indicating that the default will change to False in the next release.
  • Add logic to set cells_as_circles to True if it is None and display the deprecation warning.

For more details, open the Copilot Workspace session.

LucaMarconato and others added 3 commits August 1, 2024 21:38
This PR is AI generated (Copilot Workspace).

Related to #184

Revert the default value of `cells_as_circles` to `True` and add a deprecation warning.

* Change the default value of the `cells_as_circles` parameter to `None` in the `xenium` function.
* Add a deprecation warning indicating that the default will change to `False` in the next release.
* Add logic to set `cells_as_circles` to `True` if it is `None` and display the deprecation warning.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/scverse/spatialdata-io/issues/184?shareId=XXXX-XXXX-XXXX-XXXX).
@LucaMarconato LucaMarconato merged commit 8387b91 into main Aug 1, 2024
5 checks passed
@LucaMarconato LucaMarconato deleted the LucaMarconato/fix-xenium-default branch August 1, 2024 19:50
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.

1 participant