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

Add Support for Huron Digital Pathology Scanner Generated Images #398

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

101x4
Copy link

@101x4 101x4 commented Oct 14, 2022

This version adds supports for the huron scanner generated images.
Please close the former PR (#286), which is created by former employee from Huron, thanks.

@github-actions
Copy link

github-actions bot commented Oct 14, 2022

DCO signed off ✔️

All commits have been signed off. You have certified to the terms of the Developer Certificate of Origin, version 1.1. In particular, you certify that this contribution has not been developed using information obtained under a non-disclosure agreement or other license terms that forbid you from contributing it under the GNU Lesser General Public License, version 2.1.

@101x4 101x4 closed this Oct 14, 2022
@bgilbert
Copy link
Member

@101x4 It looks like there are two problems with your signoff lines: the signoffs need to exactly match the name and email address of the commit author, and the first two commits in the PR have not been signed off. You'll need to edit your existing commit messages with git rebase -i and then force-push.

@bgilbert
Copy link
Member

For #260.

@101x4 101x4 reopened this Oct 15, 2022
@bgilbert
Copy link
Member

@101x4 Use git rebase -i origin/main to squash all of the commits down to one commit with the correct signoff line, then git push -f.

@101x4 101x4 force-pushed the main branch 2 times, most recently from 94de38e to 0caebe7 Compare October 17, 2022 13:18
@101x4
Copy link
Author

101x4 commented Oct 17, 2022

I think the problem is that Openslide has changed a lot and our modification for HDP several years ago is no longer effective :(

Signed-off-by: Bjiao <bjiao@hurondigitalpathology.com>

update Makefile

Signed-off-by: Bjiao <bjiao@hurondigitalpathology.com>
@101x4
Copy link
Author

101x4 commented Oct 18, 2022

@bgilbert Can you help with this error: "Fetching zlib..." failed with "Failed: ${wget} -P tar "$url" (line 255)"?
I checked the Zlib website and it showed that we need to use version of 1.2.13. Thanks.
image (https://zlib.net/)

@bgilbert
Copy link
Member

It seems the previous release of zlib has been removed from its website for some reason. openslide/openslide-bin#54 should fix CI.

Signed-off-by: Bjiao <bjiao@hurondigitalpathology.com>
@101x4
Copy link
Author

101x4 commented Oct 19, 2022

@bgilbert Fetching openslide-java also failed with "Failed: ${wget} -P tar "$url" (line 255)". And how can I rerun the checks?

@bgilbert
Copy link
Member

There should be a menu entry in the triple-dots menu after you click "Details". I'm not sure if you have access to it though. I've restarted the check.

@bgilbert
Copy link
Member

Thanks for the PR! Could you submit some sample files that we can include in our sample library and in automated test cases? There's a question at the bottom of the form asking if we can redistribute the samples; we'd need that permission in order to use the slides for these purposes.

@101x4
Copy link
Author

101x4 commented Nov 14, 2022

Images uploaded, and please ignore the zip file. Thanks.

huron-admin pushed a commit to hurondp/openslide that referenced this pull request Dec 13, 2023
- Added support for Huron generated images.
- Based on the work in openslide#398

Signed-off-by: Kyle Delaney <kdelaney@hurondigitalpathology.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants