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

Improve visualization of RInChI on landing pages of reactions #60

Open
flange-ipb opened this issue Nov 24, 2023 · 1 comment
Open

Improve visualization of RInChI on landing pages of reactions #60

flange-ipb opened this issue Nov 24, 2023 · 1 comment

Comments

@flange-ipb
Copy link

Hello,

on the landing pages of reactions (e.g. reaction 1364) the RInChI and RInChIKeys are shown by expanding the RInChIKey Table.

I think the shown RInChI is missing RInChI=. Using for instance a RInChI string in rinchi_lib's rinchilib_file_text_from_rinchi requires that.

In the example 1.00.1S/C17H9BrO3/c18-14-8-4-3-7-12(14)17(21)13-9-15(19)10-5-1-2-6-11(10)16(13)20/h1-9H<>C23H19BrO3/c1-13-11-18-20(25)15-7-3-4-8-16(15)21(26)23(18,12-14(13)2)22(27)17-9-5-6-10-19(17)24/h3-10,18H,11-12H2,1-2H3<>C6H10/c1-5(2)6(3)4/h1,3H2,2,4H3!CH2Cl2/c2-1-3/h1H2/d+ should become RInChI=1.00.1S/C17H9BrO3/c18-14-8-4-3-7-12(14)17(21)13-9-15(19)10-5-1-2-6-11(10)16(13)20/h1-9H<>C23H19BrO3/c1-13-11-18-20(25)15-7-3-4-8-16(15)21(26)23(18,12-14(13)2)22(27)17-9-5-6-10-19(17)24/h3-10,18H,11-12H2,1-2H3<>C6H10/c1-5(2)6(3)4/h1,3H2,2,4H3!CH2Cl2/c2-1-3/h1H2/d+.

What about the RInChIKeys?
rinchi_lib's rinchilib_rinchikey_from_rinchi function also attaches Long-RInChIKey=, Short-RInChIKey= and Web-RInChIKey= (see example test data). InChIKey on the other hand doesn't do it like that.

Maybe @gblanke02 can point us to a "best practice style guide" for RInChI.

Best,
Frank

@gblanke02
Copy link

gblanke02 commented Nov 24, 2023 via email

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

No branches or pull requests

2 participants