Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Flex mode: Preview should include full chemical names. #4500

Open
NadezhdaPeskun opened this issue Apr 19, 2024 · 2 comments
Open

Flex mode: Preview should include full chemical names. #4500

NadezhdaPeskun opened this issue Apr 19, 2024 · 2 comments
Assignees
@mate-meszaros-epam
Labels
Content Bucket: Library content related issues Flex mode Bucket: Flex canvas related mode Priority: Medium
Milestone
Refined Backlog

Comments

@NadezhdaPeskun
Copy link

NadezhdaPeskun commented Apr 19, 2024 •
edited by AlexeyGirin
Loading

Steps to Reproduce

  1. Switch to Macro mode

  2. Open file with Peptides
    ketcher (OK).zip

  3. see preview of O peptide

Pyl O22
  1. see preview of Pyl peptide
Pyl O2

Actual behavior
One structure for O and Pyl peptides have different chemical name.

Expected behavior
Preview should include full chemical names.
as for O
Pyl O22

Also D-Pyl (name of Peptide in the Preview) should be changed to d-Pyrrolysine
image

(Agreed with BA)

By tested:

Desktop (please complete the following information):

  • OS: Windows 11
  • Browser Chrome
  • Version 123.0.6312.106 (Official Build) (64-bit)

Ketcher version
[Version 2.21.0-rc.1]
@NadezhdaPeskun NadezhdaPeskun added this to the Ketcher 2.22.0-rc.1 milestone Apr 19, 2024
@NadezhdaPeskun NadezhdaPeskun mentioned this issue Apr 19, 2024
Open
2 tasks
@AlexeyGirin AlexeyGirin added the Sequence mode Bucket: Sequence mode related issues label Apr 25, 2024
@AlexeyGirin
Copy link
Collaborator

Library name (seems - full name) should be corrected

@AlexeyGirin AlexeyGirin changed the title Sequence mode: Preview should include full chemical names. Flex mode: Preview should include full chemical names. May 6, 2024
@AlexeyGirin AlexeyGirin added Flex mode Bucket: Flex canvas related mode Content Bucket: Library content related issues and removed Sequence mode Bucket: Sequence mode related issues labels May 6, 2024
@mate-meszaros-epam mate-meszaros-epam self-assigned this May 14, 2024
@mate-meszaros-epam
Copy link
Contributor

This issue is a database issue. The code is trying to use monomer full name but some cases it is loaded incorrectly.
monomerFullName

@AlexeyGirin AlexeyGirin removed the bug label Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mate-meszaros-epam mate-meszaros-epam

Labels
Content Bucket: Library content related issues Flex mode Bucket: Flex canvas related mode Priority: Medium
Projects
None yet
Milestone
Refined Backlog
Development

No branches or pull requests
4 participants
@rrodionov91 @AlexeyGirin @mate-meszaros-epam @NadezhdaPeskun