Uh oh!
There was an error while loading. Please reload this page.
schema and dataclasses (wip) - #40
Conversation
codecov-commenter
commented
Oct 22, 2022
Codecov Report
Additional details and impacted files@@ Coverage Diff @@## feature/transform #40 +/- ##
=================================================
Coverage 0.00% 0.00% =================================================
Files 14 15 +1 Lines 1119 1158 +39 =================================================
- Misses 1119 1158 +39
Flags with carried forward coverage won't be shown. Click here to find out more.
|
* Create design_doc.md * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * reviewed design doc, updated with pseudocode * renamed NgffStore -> NGFFStore * Update README.md * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * @giovp comments * Update docs/design_doc.md Co-authored-by: Giovanni Palla <25887487+giovp@users.noreply.github.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Luca Marconato <2664412+LucaMarconato@users.noreply.github.com> Co-authored-by: Giovanni Palla <25887487+giovp@users.noreply.github.com>
giovp
commented
Nov 12, 2022
polygons IO only works with |
open questions:
|
| ) -> AnnData: | ||
| # TODO: AnnData(obsm={"spatial": data}) doesn't work, shall we change? | ||
| adata = AnnData(np.empty(shape=data.shape), obsm={"spatial": data}) |
There was a problem hiding this comment.
@ivirshup this AnnData(obsm={"spatial": data}) doesn't work. Quite important if we want to save shapes/points in anndata with coordinates in obsm (and X potentially None ). What should we do?
| # TODO: do not save copy but pop transform? | ||
| shapes_copy = shapes.copy() | ||
| shapes_copy.uns.pop("transform") | ||
| write_adata(sub_group, name, shapes_copy) |
There was a problem hiding this comment.
basically if adata.uns["transform"] = spatialdata.Transform(...) it can't ofc be serialized so it needs to be popped. However, that would change the in-memory representation. Can only copy or someone has better idea?
giovp
commented
Nov 15, 2022
zarr group hierarchy of elements: ImageLabelsShapesPolygonsTableand the let's use this for discussion, think we can improve a bit. |
Uh oh!
There was an error while loading. Please reload this page.
No description provided.