Fix's issue with OpenSlide downsampling alignment and Improves downsa…

Philbrick commented 3 years ago

…mpling perf for images with only high resolution scans

Adds progressiveDownsample option. Supported for all image generation paths. (Bilinear & Nearest neighbor). Alters downsampling algorithm to progressively downsample images instead of downsampling from nearest image in the scanned slide pyramid.

Fix alignment issue caused by rounding errors in openslide
improves performance for pyramid generation for slides with only high resolution layers.
Added end-to-end test cases for (Bilinear & Nearest neighbor)
unit test for new code paths (dicom_file_region_reader.cpp & zlibWrapper).

danielbeaudreau commented 3 years ago

This PR is huge, is there any way you can split this into a series of smaller ones ideally < 500 lines each?

Philbrick commented 3 years ago

Breaking up and submitting in smaller parts. Leaving open until fully added.

Philbrick commented 3 years ago

Changes integrated.

GoogleCloudPlatform / wsi-to-dicom-converter

Fix's issue with OpenSlide downsampling alignment and Improves downsa… #59