Skip to content

Add support for loading selection/highlight from CXSMILES and MDL Mol… - #1246

Merged
egonw merged 1 commit into
mainfrom
chemobjselection
Nov 30, 2025
Merged

Add support for loading selection/highlight from CXSMILES and MDL Mol…#1246
egonw merged 1 commit into
mainfrom
chemobjselection

Conversation

@johnmay

@johnmay johnmay commented Nov 10, 2025

Copy link
Copy Markdown
Member

…files.

  • We have an existing interface IChemObjectSelection which support highlighting when rendering. These needed to be moved to the cdk-interfaces module.
  • In CXSMILES we can use the EPAM extension of ha: and hb: to load highlighted bonds/atoms
  • MDLV3000 has HILITE collection which we can use
  • Fix a bug with the writing of MDLV3000 stereo groups, the first digit needs to be the number of things in the set

…files.

- We have an existing interface IChemObjectSelection which support highlighting when rendering. These needed to be
moved to the cdk-interfaces module.
- In CXSMILES we can use the EPAM extension of ha: and hb: to load highlighted bonds/atoms
- MDLV3000 has HILITE collection which we can use
- Fix a bug with the writing of MDLV3000 stereo groups, the first digit needs to be the number of things in the set
@sonarqubecloud

Copy link
Copy Markdown

@johnmay

johnmay commented Nov 11, 2025

Copy link
Copy Markdown
Member Author

@egonw This is good to go

@johnmay

johnmay commented Nov 30, 2025

Copy link
Copy Markdown
Member Author

@egonw Could you merge this in at some point - I saw Steinbeck Labs' cheminformatics-microservice have also added support in a jank way (a seperate MDLHiliteParser). It make's much more sense to have it pushed down in the MDL and CXSMILES readers as it is here - thanks.

@egonw

egonw commented Nov 30, 2025

Copy link
Copy Markdown
Member

Could you merge this in at some point

@johnmay, sorry, it have been three tough months... lost track on this one.

@egonw
egonw merged commit 602c911 into main Nov 30, 2025
10 of 12 checks passed
@johnmay
johnmay deleted the chemobjselection branch March 3, 2026 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants