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

SMILES/SMARTS import: Files with S-Group Properties cause an error #1040

Closed
Zhirnoff opened this issue Feb 16, 2023 · 1 comment · Fixed by #1201
Closed

SMILES/SMARTS import: Files with S-Group Properties cause an error #1040

Zhirnoff opened this issue Feb 16, 2023 · 1 comment · Fixed by #1201

Comments

@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Feb 16, 2023

Steps to Reproduce

  1. Open Ketcher
  2. Paste CCC |SgD:1:atropisomer:info2::::|

Expected behavior
SMILES/SMARTS files with S-Group Properties loading without error
2023-02-16_22h22_36
2023-02-16_21h39_52

Actual behavior
SMILES/SMARTS files with S-Group Properties loading with convert error

Attachments
2023-02-16_21h36_28

Indigo/Bingo version
Indigo Toolkit
Version 1.10.0-rc.3.0-g8e56c89e5-x86_64-linux-gnu-11.2.1

#183928124

@Zhirnoff Zhirnoff added the Bug label Feb 16, 2023
@Zhirnoff Zhirnoff added this to the Indigo-1.11.0-rc.1 milestone Feb 16, 2023
@Zhirnoff Zhirnoff added this to Product Backlog in Indigo via automation Feb 16, 2023
@even1024 even1024 moved this from Product Backlog to In progress in Indigo Feb 22, 2023
@even1024 even1024 moved this from In progress to To Do in Indigo Mar 7, 2023
@GuskoMaksim GuskoMaksim removed their assignment Jul 16, 2023
@even1024 even1024 self-assigned this Jul 17, 2023
@even1024 even1024 moved this from To Do to In progress in Indigo Jul 18, 2023
@even1024 even1024 closed this as completed Aug 1, 2023
Indigo automation moved this from In progress to Done Aug 1, 2023
@Zhirnoff
Copy link
Collaborator Author

Autotest written.

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

Successfully merging a pull request may close this issue.

4 participants