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

Macro: Provide save/open ket and molv3000 ketcher API for macromolecules #3531

Closed
10 tasks done
rrodionov91 opened this issue Nov 2, 2023 · 1 comment · Fixed by #3637
Closed
10 tasks done

Macro: Provide save/open ket and molv3000 ketcher API for macromolecules #3531

rrodionov91 opened this issue Nov 2, 2023 · 1 comment · Fixed by #3637

Comments

@rrodionov91
Copy link
Collaborator

rrodionov91 commented Nov 2, 2023

Background
It needs to have an ability to save/open ket and molv3000 files through ketcher API. It should be available in global "ketcher" variable similar as for micromolecules.

The following API methods should be changed/implemented:

  • Implement getKet for Macromolecules
  • Implement getMol for Macromolecules
  • Implement setMolecule for Macromolecules
  • Implement addFragment for Macromolecules
  • layout should throw an Error 'layout is not available' in macro mode
  • recognize should throw an Error 'recognize is not available' in macro mode
  • generateImage should work with macromolecules; Most likely, output will be different from molecules drawn in Ketcher
  • setSettings – when executed does nothing, as there are no settings for macro mode yet (may affect preview for monomers)
  • containsReaction – no actual changes in functionality; returns true if structure has a reaction in micro mode
  • getSmiles, getCdx, ... and other formats, except getKet and getMolfile – throw Error
@rrodionov91 rrodionov91 added this to the Macromolecules Milestone 2 milestone Nov 2, 2023
@rrodionov91 rrodionov91 changed the title Macro: provide ketcher API for macromolecules Macro: Provide ketcher API for macromolecules Nov 2, 2023
@rrodionov91 rrodionov91 changed the title Macro: Provide ketcher API for macromolecules Macro: Provide save/open ket and molv3000 ketcher API for macromolecules Nov 10, 2023
Nitvex added a commit that referenced this issue Dec 4, 2023
Nitvex added a commit that referenced this issue Dec 4, 2023
Nitvex added a commit that referenced this issue Dec 4, 2023
Nitvex added a commit that referenced this issue Dec 4, 2023
Nitvex added a commit that referenced this issue Dec 4, 2023
Nitvex added a commit that referenced this issue Dec 4, 2023
Nitvex added a commit that referenced this issue Dec 4, 2023
Nitvex added a commit that referenced this issue Dec 4, 2023
…romolecules (#3637)

* #3531 – added getKet, getMol, setMolecule for macromode

* #3531 – added tests for ketcher API for macro mode

* #3531 – updated documentation for Ketcher API

* #3531 – fixed tests
@Zhirnoff
Copy link
Collaborator

Manual tests added to automation table

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