Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Already on GitHub? Sign in to your account
Support PreciPoint VMIC format #168
Comments
bgilbert
added enhancement format-new
labels
Dec 7, 2015
Markus-PP
commented
Mar 9, 2016
|
I am ready to work on this. Has anyone else given the matter some thoughts yet ? Obviously it requires a ZIP library. After some tests, I concluded that libzip does what we need. Are there any objections against including libzip ? Will there be other slides with images in containers supported by libzip ? |
|
Awesome, thanks for looking into this! When writing your code, please ensure that you do not use any information obtained under a non-disclosure agreement. If you use NDA-protected material, we will not be able to merge your changes. libzip would work, but wouldn't be my first choice. libgsf will read both ZIP and OLE files, which might be useful someday for Zeiss CZI. If you do use libzip, note that version 1.0 is not yet available in many supported Linux distributions. I'm not currently aware of any other primary formats that use ZIP containers, but I'd suggest making an Do you have sample VMIC files? |
|
Does VMIC use ZIP64?
|
Markus-PP
commented
Mar 9, 2016
|
Thanks for fast reply ! I have approval and support of the PreciPoint team so i don't see any legal issue here. I do have sample files, and I know ZIP64 is actually required. Thanks for hinting that older versions don't support it - I did test runs with libzip 1.1.2 (the newest) which works like charm on VMIC, will have to check if libzip 0.11.2 and libgsf do the job, too. Also planning to set up an openslide-vendor-deepzoom.c for generalizing purposes. Will probably have more questions but not at this point. Much studying is still required. [My distribution, OpenSuSE 13.2, has libzip 0.11.2 out-of-the-box, updateable from repo to 1.1.2; and libgsf 1.14.30, updateable to 1.14.33] |
|
If older versions of a library simply reject ZIP64, that's fine. If they crash or behave incorrectly (as libgsf does) we will need to conditionally enable ZIP support. Are you able to share the sample files under CC0? We strongly prefer that new drivers come with freely-redistributable samples. (Indeed, we can't create automated test cases without them.) Some background on the NDA stuff: we have encountered vendors who have offered to provide documentation for an OpenSlide driver, but only under restrictions which are unworkable for an open-source project. Therefore, to avoid problems, we've been planning to require contributors to certify that they have the right to contribute the code they are submitting under OpenSlide's license. It's the same process used in Linux kernel development: each Git commit message will need to include a |
Markus-PP
commented
Apr 24, 2016
|
PreciPoint team have agreed on providing some VMIC- example files for public domain. https://drive.google.com/file/d/0BwWPfDJXmk2gbm5ROU5TZUNxVms/view?usp=sharing I have written some code which is in experimental stage and somewhat working, although not ready for inclusion in the library. Shall I discuss it here or rather in the mailing list instead ? |
|
Thanks for the slide files! Note that public domain isn't legally valid in all countries. Would it be possible for the samples to be licensed under CC0? (CC0 is essentially public domain plus a permissive license for places where public domain doesn't exist.) If your code isn't ready for a pull request, I'd suggest discussing it on the mailing list. |
Markus-PP
commented
Sep 19, 2016
|
Yes, of course they can be licensed under CC0, as well, since the PreciPoint team ensured they are devoid of third-party rights. Is perhaps any official statement needed ? A commit is under way, but before doing so i'm going to send a reply on the mailing list. |
Markus-PP
commented
Sep 26, 2016
|
Repository https://github.com/Markus-PP/openslide-vmic has a functional driver. |
|
@Markus-PP For now, if you could post an official statement in this thread (or send it to me privately), I could add the samples to openslide-testdata. That would allow VMIC tests to be added to the test suite. Eventually we'll have a web form for collecting CC0 signoffs, and at that point I'll ask you to complete it. |
Markus-PP
commented
Sep 29, 2016
|
I hereby confirm that the PreciPoint team agrees to release the above mentionend 3 slides under CC0 and that, to the best of my knowledge, they are devoid of any third party rights. |
|
Thank you very much! I just noticed that each sample includes a copy of OpenSeadragon, which can't be licensed under CC0 because it is subject to third-party copyright. OpenSeadragon is BSD-licensed and the license information is properly included in the slide files, so we can redistribute the samples, but I'll need to mark them as a mix of CC0 and BSD. In each of the inner ZIP files, there is a small number of members with a |
|
Also:
|
Markus-PP
commented
Oct 3, 2016
|
To the first question: I guess, at the moment it's probably best to remove the seadragon references, because they are not needed for any of the viewer applications. The original goal was to provide the customer a way to easily unpack and view the slide without extra software. The bogus jpg.bmp files, if I remember correctly, were created during testing and they have no purpose. I'll talk with the team next week to know for sure, Then, I can also fetch the textual description, together with the seadragon-free slides. |
|
Great, I'll credit PreciPoint and rename the files. We should avoid modifying the samples after they are written by your software; otherwise they won't be accurate samples. If current versions of your software no longer embed OpenSeadragon, then I'd be in favor of generating some new scans. Otherwise, let's use unmodified files and just list both licenses. (Mixed licensing is not uncommon in open-source projects.) |
bgilbert commentedDec 7, 2015
Apparently it's roughly Deep Zoom inside ZIP.