Skip to content

Add ability to display names under compounds with RDKit#79

Merged
Acylation merged 2 commits into
Acylation:mainfrom
mimvoid:add-molecule-names
Aug 14, 2025
Merged

Add ability to display names under compounds with RDKit#79
Acylation merged 2 commits into
Acylation:mainfrom
mimvoid:add-molecule-names

Conversation

@mimvoid

@mimvoid mimvoid commented Dec 15, 2024

Copy link
Copy Markdown
Contributor

Use RDKit.js's ability to detect a compound's _Name property and display it in the legend. Closes #75.

I looked through smilesDrawer's source code and unfortunately could not find any equivalent. As far as I can tell, it's only possible with RDKit.

Syntax:

C1([Cl])=CC=CC(C(=O)O[OH])=C1 mCPBA

Result:

2024-12-14_18-32-46

@Acylation Acylation added the category/new feature The feature will be added label Dec 15, 2024
@Acylation

Copy link
Copy Markdown
Owner

Thanks for figuring this out! I'm struggling with my upcoming finals and will possibly reviewing these next weekend.

@mimvoid

mimvoid commented Dec 15, 2024

Copy link
Copy Markdown
Contributor Author

Thanks for figuring this out! I'm struggling with my upcoming finals and will possibly reviewing these next weekend.

It's also finals week for me haha. Good luck!

@thalesdapraia-boop

Copy link
Copy Markdown

how can I add this to my obsidian?

@thalesdapraia-boop

Copy link
Copy Markdown

how can I add this to my obsidian?

rn I'm using tables to identify them but it would be so nice to have the names under the molecules

@mimvoid

mimvoid commented Aug 8, 2025

Copy link
Copy Markdown
Contributor Author

how can I add this to my obsidian?

I'm not sure when this pull request will be accepted, but for now, you can try installing the plugin from my fork.

@Acylation, if you aren't maintaining this plugin much anymore, I'm happy to continue development on my fork.

@mimvoid mimvoid force-pushed the add-molecule-names branch from 1f74a70 to 7b13bd3 Compare August 8, 2025 00:52
@Acylation

Copy link
Copy Markdown
Owner

Allow me some time to fix the aborted call of app instance which is no longer supported by Obsidian.

@Acylation Acylation merged commit 7885028 into Acylation:main Aug 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category/new feature The feature will be added

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Assign a number or name under the structure

3 participants