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

Only one molecule from the input SMILES file is processed... #30

Open
UnixJunkie opened this issue Nov 12, 2018 · 7 comments
Open

Only one molecule from the input SMILES file is processed... #30

UnixJunkie opened this issue Nov 12, 2018 · 7 comments
Labels

Comments

@UnixJunkie
Copy link

Hello,

molvs standardize -i smi input.smi  -o smi -O output.smi

The input file has several molecules, the output a single one.
Is this the expected behavior?

@UnixJunkie
Copy link
Author

UnixJunkie commented Nov 12, 2018

I installed via pip.

pip install molvs

@UnixJunkie
Copy link
Author

this is probably the file that needs a fix:
https://github.com/mcs07/MolVS/blob/master/molvs/cli.py

@mcs07
Copy link
Owner

mcs07 commented Feb 5, 2019

Yes that seems to be the case. Also it looks like SDF files will only process the first molecule.

@mcs07 mcs07 added the bug label Feb 5, 2019
@UnixJunkie
Copy link
Author

also, output to sdf file is not supported (it creates a mol file instead)

@UnixJunkie
Copy link
Author

I'll send a patch for some of these.

@sgujjaABM
Copy link

Hi, I am having the same issue with an SDF file. Wondering if there's a fix?
Thanks

@UnixJunkie
Copy link
Author

I sent a PR with a fix.
Use the molVS that is inside of rdkit, then you should be good to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants