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

MDL Molfile v3000: when opening files containing 'Salts an Solvents', names are truncated and abbreviation is expanded #965

Closed
Zhirnoff opened this issue Jan 4, 2023 · 1 comment · Fixed by #1125
Assignees
Labels
Projects

Comments

@Zhirnoff
Copy link
Collaborator

Zhirnoff commented Jan 4, 2023

Steps to Reproduce

  1. Launch Ketcher
  2. Put any "Salts and Solvents" whose names contain spaces on canvas
  3. Save as MDL Mol v3000
  4. Open or Paste from Clipboard
    Note: In MDL Molfile v3000 the name of the superatoms must be quoted if there are spaces.
    Abbreviations are expanded because the name changes and they become unidentified.
    2023-01-04_18h06_31
    MicrosoftTeams-image (1)

Expected behavior
When opening files containing 'Salts an Solvents', the names are not truncated at the first space in the name and the abbreviation is parsed as it was saved

Actual behavior
When opening files containing 'Salts an Solvents', the names are truncated at the first space in the name and the abbreviation is parsed expanded

Attachments

2023-01-04_18h04_07.mp4

Indigo/Bingo version
Indigo Toolkit
Version 1.9.0-rc.2.0-g40c4e3793-x86_64-linux-gnu-11.2.1

Additional context
Related Ketcher issue: epam/ketcher#2143

@Zhirnoff Zhirnoff added this to the indigo-1.9.0 milestone Jan 4, 2023
@Zhirnoff Zhirnoff added this to Product Backlog in Indigo via automation Jan 4, 2023
@vanoprenko vanoprenko changed the title MDL Molfile v3000: When opening files containing 'Salts an Solvents', the names are truncated at the first space in the name and the abbreviation is parsed expanded MDL Molfile v3000: when opening files containing 'Salts an Solvents', names are truncated and abbreviation is expanded Jan 5, 2023
@even1024 even1024 closed this as completed Jan 6, 2023
Indigo automation moved this from Product Backlog to Done Jan 6, 2023
@even1024 even1024 moved this from Done to Product Backlog in Indigo Jan 9, 2023
@even1024 even1024 reopened this Jan 9, 2023
@even1024 even1024 removed the rc3 label Jan 9, 2023
Indigo automation moved this from Product Backlog to Done Feb 21, 2023
even1024 added a commit that referenced this issue Feb 27, 2023
…nts', names are truncated and abbreviation is expanded (#1046)

* molecule label

* molecule label

---------

Co-authored-by: Roman Porozhnetov <roman_porozhnetov@epam.com>
@Zhirnoff
Copy link
Collaborator Author

Zhirnoff commented May 4, 2023

The second half of the Salts and Solvents name is lost and quotes appear in the first part of the name.

2023-05-04_20h23_40.mp4

Wadym pushed a commit that referenced this issue May 8, 2023
…nts', names are truncated and abbreviation is expanded (#1125)
Wadym pushed a commit that referenced this issue May 9, 2023
…nts', names are truncated and abbreviation is expanded (#1125)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Indigo
  
Done
2 participants