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

Extra space in RXN v3 ? #111

Closed
lpatiny opened this issue Jul 25, 2024 · 2 comments
Closed

Extra space in RXN v3 ? #111

lpatiny opened this issue Jul 25, 2024 · 2 comments

Comments

@lpatiny
Copy link

lpatiny commented Jul 25, 2024

It seems that when we get the RXN v3000 there is an extra space at the beginning of the first line.

@thsa

$RXN V3000


OCL_RXN_V1.0:gJX@@eKU@@ gGQHDHaImfh@!defH@DAIfUVjj`@#IGiYP qDir IjAKiqeP#!R@Fq?[@@S_qL]x`H !RbOs~@Hc}bOrH?QulH` !R?g~w?Xc}m?s~_xc|m?vH?Xc}GYk\##
 M  V30 BEGIN REACTANT
M  V30 BEGIN CTAB
M  V30 COUNTS 6 5 0 0 0
M  V30 BEGIN ATOM
M  V30 1 C 2.369 -9.0416 0 2
M  V30 2 C 3.2351 -8.5416 0 3
M  V30 3 C 4.1011 -9.0416 0 4

image

@targos
Copy link
Contributor

targos commented Jul 25, 2024

Seems to be a bug here:

theWriter.write(String.format("M V30 COUNTS %d %d"+NL,rcnt,pcnt));

It ends up writing a single space character instead of a COUNTS line.

@targos
Copy link
Contributor

targos commented Jul 25, 2024

@lpatiny The bug is only in the JS version. You can close this issue.

targos added a commit to cheminfo/openchemlib-js that referenced this issue Jul 25, 2024
@lpatiny lpatiny closed this as completed Jul 25, 2024
targos added a commit to cheminfo/openchemlib-js that referenced this issue Jul 25, 2024
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