date.parse() in nlp_core

I tried all the readme examples, and I got very mixed results. Did something in the library break or am I doing something wrong?

nlp.text('She sells seashells').to_past()
// An array containing the object shown below. Object contains text from original string.

nlp.noun("dinosaur").pluralize();
// This works

nlp.verb("speak").conjugate();
// This works

nlp.text('She sells seashells').negate()
// Same issue as above

nlp.sentence('I fed the dog').replace('the [Noun]', 'the cat')
// Same issue as above

nlp.text("Tony Hawk did a kickflip").people();
// This works

nlp.person("Tony Hawk").article();
// "a" instead of "he" 

nlp.value("five hundred and sixty").number;
// This works

I’m using Chrome Version 49.0.2623.112 (64-bit) on OS X I got the same results in Node.js 5.10.1

[spencermountain]

hey KahWee, yeah, this is the softest part of nlp_compromise by a long shot. That’s a great, (and brutal!) example 😉

So for sure it should remain an array of results, you can mention two independent dates per sentence, but the result format is pretty lame.

My intention with the format was to support strings like "in june", without assuming a particular year, likewise for hour/second/etc information - returning a date object forces choices about which year/second it is, even when you don’t have information about it.

The second thing is date intervals, like "june 5th to 6th", which i originally supported but stepped-back from.

The third thing is repeating dates, like "every tuesday", which needs some kind of result format that respects the wonky calendar-systems, leap-years etc.

So you can see how limp-wristed the date-parsing method, and result-format is in current form. What it is good at though, is identifying which part of a text is a date. Maybe date.parse() should be removed completely, or created as a plugin, or something like this. I just added a URL.parse() method with the same feel, so maybe we’re onto something.

Anybody with ideas or ambitious with this, that’d be sweet cheers!

[stanonyime]

On Chrome, i’m getting errors on:

nlp.value("I married April for the 2nd time on June 5th 1998 ").date()

The error is:

VM235:1 Uncaught TypeError: nlp.value(...).date is not a function