Skip to content

Add H&E data loader#40

Open
heylf wants to merge 3 commits into
mainfrom
add-dataloader-he
Open

Add H&E data loader#40
heylf wants to merge 3 commits into
mainfrom
add-dataloader-he

Conversation

@heylf
Copy link
Copy Markdown
Contributor

@heylf heylf commented May 19, 2026

Describe your changes

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!

rcannood

This comment was marked as outdated.

Copy link
Copy Markdown
Member

@rcannood rcannood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait no, there is something wrong with the config. In the CI:

Reading file 'src/datasets/loaders/hae/config.vsh.yaml' failed

Can you run viash test src/datasets/loaders/hae/config.vsh.yaml ? This should tell you what's wrong with the dataset loader

@heylf
Copy link
Copy Markdown
Contributor Author

heylf commented May 19, 2026

Yeah. It is a bit for discussion now how we deal with H&E. I think it would be great now to have a metric that utlized this data type. I think I made a mistake to implement a separate workflow for the data type.

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