ChemDataExtractor icon indicating copy to clipboard operation
ChemDataExtractor copied to clipboard

Fix tokenizer problem with commas inside NMR peaks

Open JeffersonH44 opened this issue 7 years ago • 2 comments

Hello,

I found a problem while you are reading NMR peaks when you have no whitespace between two elements, like this:

9.73 (s,1H)

This is parsed as a peak with an assignment instead of a peak with a multiplicity and a number, I also add some cases for other contexts when you have some coupling units.

JeffersonH44 avatar Aug 11 '17 08:08 JeffersonH44

Sorry I've taken so long to review this - there's some great stuff here. But could you create separate branches for each individual change so they can have separate, more focused Pull Requests?

That way you can also continue developing on your master branch and it won't appear in the pull request. I think you'll need to create a new branch from the last commit before your changes, then you can cherry pick some of your own commits into the branch.

Some help documentation here: https://help.github.com/categories/collaborating-with-issues-and-pull-requests/ https://git-scm.com/docs/git-cherry-pick

mcs07 avatar Sep 03 '17 21:09 mcs07

Ok, I'll do that

JeffersonH44 avatar Sep 03 '17 21:09 JeffersonH44