Skip to content

Refining base repo#5

Open
heylf wants to merge 12 commits intoopenproblems-bio:mainfrom
heylf:main
Open

Refining base repo#5
heylf wants to merge 12 commits intoopenproblems-bio:mainfrom
heylf:main

Conversation

@heylf
Copy link
Copy Markdown

@heylf heylf commented Apr 22, 2026

Describe your changes

Refining the base repo. Filling in also the data_processor stuff.

Checklist before requesting a review

  • I have performed a self-review of my code

  • Check the correct box. Does this PR contain:

    • Breaking changes
    • New functionality
    • Major changes
    • Minor changes
    • Bug fixes
  • Proposed changes are described in the CHANGELOG.md

  • CI Tests succeed and look good!

@heylf
Copy link
Copy Markdown
Author

heylf commented Apr 22, 2026

I still try to figure out the cli of openproblems. So I did the PR without proper local testing.

@heylf heylf changed the title changes many files Refining base repo Apr 22, 2026
@heylf
Copy link
Copy Markdown
Author

heylf commented Apr 23, 2026

Seems to fail now at the point again where spatialdata object is created for cellpose. So the dataprocessor is kinda done.

@rcannood
Copy link
Copy Markdown
Member

For process_dataset, the logs say:

AssertionError: File 'output_spatial_dataset.zarr' is missing images['image']

Is it expected that the output doesn't have an image, as requested by the api? https://github.com/openproblems-bio/task_spatial_segmentation

@heylf
Copy link
Copy Markdown
Author

heylf commented Apr 24, 2026

Yeah, I did some weird stuff. I made now the proper change and commented out, because so far I dont fully know if the spatialdata needs standard processing. Only the single cell data needs.

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