epam / epam/Indigo

Display placeholder for corrupted Images in CDXML files

Open
#2,325 0 comments 0 reactions 0 assignees View on GitHub
MMPK Priority: Low
Dominant language
C++
Stars
406
Forks
134
Avg merge
2d 11h
Merged PRs (30d)
24

Description

**Background**
When we load a corrupted CDXML file containing an image, the system either fails to display the image entirely or shows a partial image. In some cases, the borders of the image become visible when selected with the selection tool, even though the image itself is not displayed correctly.
![2024-09-06_12h25_38](https://github.com/user-attachments/assets/9e291c0e-5d59-4b3f-968d-1f39d9ed6771)
Corrupted file: [corrupted image file.zip](https://github.com/user-attachments/files/16905859/corrupted.image.file.zip)

**Solution**
**TBD** - After discussion with the team, @vanoprenko, and @AlexanderSavelyev, we decided the following: The system should detect when an image is corrupted in the CDXML file and display a placeholder or error icon indicating that the image cannot be loaded. This will provide clarity to the user, rather than displaying partial or no images at all.

**Note:** We have similar issue https://github.com/epam/ketcher/issues/5146
Issue found during testing: https://github.com/epam/Indigo/issues/2028
Issue on Ketcher side: https://github.com/epam/ketcher/issues/5446

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reproducing the problem with the attached corrupted image file.zip and trace the CDXML image-loading path. The issue is done when corrupted images are detected consistently and the UI displays a placeholder or error icon instead of a partial or missing image.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
computer-graphics
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.