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.

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

Unclear/inconsistent handling of absolute/relative stereochemistry #290

Closed
sharped-rsc opened this issue Feb 4, 2021 · 8 comments · Fixed by #560
Closed

Unclear/inconsistent handling of absolute/relative stereochemistry #290

sharped-rsc opened this issue Feb 4, 2021 · 8 comments · Fixed by #560

Comments

@sharped-rsc
Copy link

Hi,

I'm trying to find a way to allow users to draw chemical structures with absolute stereochemistry in ketcher and get the structures out as mol files - I've so far not found how to do this so I would appreciate any help or advice you are able to provide.

While trying to find the answer I've explored the Ketcher 2.0 demo page
https://lifescience.opensource.epam.com/ketcher/demo.html) current spec:

I would appreciate any help you are able to provide.

Dave

@AlexanderSavelyev
Copy link
Contributor

@sharped-rsc could you please clarify your question. Do you want to add enhanced stereo information with ABS labels for the structures?

@sharped-rsc
Copy link
Author

HI @AlexanderSavelyev , thanks. So I have a lot of feedback that I was originally going to put in the card - hence the slightly odd title.
image
But principally - if a user draws L-alanine into the sketcher - I'd like to be able to have the ability to get the molfile from that and use that in a search. As a user I'd expect the search (and by implication the molfile) to be specifically about the enantiomer that I've drawn and not either enantiomer - ie the chiral flag should be set in the molfile)

In terms of my other feedback which is mainly UX and related points around the stereochemistry - would it make sense to add them as comments to this card, open a new card, or send an email to a mailing list?

Thanks for your help.

Dave

@sharped-rsc
Copy link
Author

Hi @AlexanderSavelyev,
Was my explanation sufficient and clear, or do you need more details?

Kind regards,
Dave

@AlexanderSavelyev
Copy link
Contributor

Hi @sharped-rsc

Yes the explanation is clear. Right now we are in progress of implementing enhanced stereochemistry support, and this case should be covered by #56 epic

@sharped-rsc
Copy link
Author

Thanks for the update. Would it be ok if I add some comments to some of the cards in that epic?

@AndreiMazol
Copy link
Contributor

@sharped-rsc I have linked your issue to #56 so no needs to add any additional information to the epic.

@AndreiMazol AndreiMazol added this to the Enhanced Stereochemistry milestone Mar 9, 2021
@AndreiMazol AndreiMazol modified the milestones: Enhanced Stereochemistry, Release 2.3.0 Apr 21, 2021
@Svetlana-Asinovskaia
Copy link
Collaborator

Mol V2000 doesn’t support enhanced stereochemistry. In case a user saves a structure with the enhanced stereochemistry features in Molfile V2000 the warning “Structure contains enhanced stereochemistry features. Information will be partly lost” appears.

After saving in Molfile V2000 Stereo Flags are displayed as follows:

  • If all saved structures had ABS (Chiral) Flags, they will be present on the canvas after the saved file is opened.
  • If one of the saved structures had the ABS (Chiral) Flag, then after opening the saved file, all structures will be displayed with the ABS (chiral) Flags.
  • If none of the structures had the ABS (Chiral) Flag, then after opening the saved file, all structures will be displayed with the AND Enantiomer Flags.

The issue was discussed with @AlexanderSavelyev

@AndreiMazol AndreiMazol linked a pull request Jun 2, 2021 that will close this issue
@Svetlana-Asinovskaia
Copy link
Collaborator

Release/ 2.3 The issue is verified and closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants