MRADUL TRIPATHI
MRADUL TRIPATHI
Hi @bsayak03 Can i work in this?
Hi @joelbyler 👋 I’ve submitted a fix for issue https://github.com/microsoft/presidio/issues/1731 (handling XA modality multi-frame grayscale DICOMs). The changes add a _dicom_np_to_pil helper and improve handling in _get_most_common_pixel_value and _set_bbox_color to...
Hi @omri374 @joelbyler 👋 I’ve submitted a fix for issue #1731 (handling XA modality multi-frame grayscale DICOMs). The changes add a `_dicom_np_to_pil` helper and improve handling in `_get_most_common_pixel_value` and `_set_bbox_color`...
Hi @omri374 Thanks for the feedback earlier. I've added the requested unit test for multi-frame grayscale DICOM handling. All tests are passing locally Please let me know if you’d like...
Thanks @joelbyler Agreed — multi-frame redaction across all frames would definitely be valuable, and I agree it should be tracked as a separate enhancement issue. When you test it out,...
Hi @joelbyler Thanks a lot for reviewing and confirming that the exception is resolved 🙏 Since this PR addresses the reported issue (#1731) and the separate multi-frame enhancement is now...
Thanks a lot @joelbyler for confirming that the fix works for your use case 🙏 Hi @omri374 👋 Since this PR resolves the reported issue (#1731) and tests are passing...
Hi @omri374 👋 Just following up on this PR — since it resolves issue #1731 and tests are passing locally, could you please take a look and let me know...
Hi @omri374 👋 I noticed that several CI jobs (linting + test_image_redactor) are failing after my changes. Could you please guide me on what adjustments are needed to get all...
@joelbyler can you help me with that??