search

imi-bigpicture / wsidicomizer

Python library for converting WSI files to DICOM
Apache License 2.0
54 stars 7 forks source link

Support for Philips isyntax images #61

Open shaneahmed opened 1 year ago

shaneahmed commented 1 year ago

It would be useful to add support for Philips isyntax images. Currently, there is lack of open source libraries which support conversion from isyntax. These libraries also require installation of Philips SDK. It would be really helpful if there is a way to support this with an open licence.

erikogabrielsson commented 1 year ago

Hi @shaneahmed

There is an issue open on openslide for adding the iSyntax-implementation from slidescape. This implementation does not use the SDK and could be a good solution also for converting do DICOM. As this is actively worked on we will monitor the development and, when/if done, implement any needed changes to wsidicomizer.

shaneahmed commented 1 year ago

Hi @shaneahmed

There is an issue open on openslide for adding the iSyntax-implementation from slidescape. This implementation does not use the SDK and could be a good solution also for converting do DICOM. As this is actively worked on we will monitor the development and, when/if done, implement any needed changes to wsidicomizer.

The issue with slidescape is that it uses GPL-3 license. Therefore, I don't see an integration in the near future unless the license is updated. But I agree let's keep an eye on this.

erikogabrielsson commented 1 year ago

The author is willing to license the isyntax part under a more permissive license: https://github.com/openslide/openslide/issues/433#issuecomment-1488938986

shaneahmed commented 1 year ago

The author is willing to license the isyntax part under a more permissive license: https://github.com/openslide/openslide/issues/433#issuecomment-1488938986

That would be great!

erikogabrielsson commented 5 months ago

Initial support for converting isyntax files is available in the isyntax branch. This uses the python wrapper pyisyntax and libisyntax. It seems to work OK for isyntax but there are some problems with i2syntax. Only the base level is read from isyntax, but a pyramid can be recreated using the add_missing_levels-parameter.

erikogabrielsson commented 2 months ago

@shaneahmed Have you been able to test the isyntax support?

shaneahmed commented 1 month ago

@skiros-habib Please can you check this as discussed?