search

jflanigan / jamr

JAMR Parser and Generator
BSD 2-Clause "Simplified" License
193 stars 50 forks source link

Fixing parse script bug in NER output reading #7

Closed jkkummerfeld closed 9 years ago

jkkummerfeld commented 9 years ago

I realised this was the source of the problem I mailed you about - the NER output processing part of the PARSE script was using a negative index to head, which is not supported by BSD head (used on OS X). I've changed the scala script instead (making it more imperative, but getting the job done).

jflanigan commented 9 years ago

Thanks for tracking down what caused this error. I'm fixing this in the main branch in a slightly different way (using awk instead of head, which should be supported on OS X).

jkkummerfeld commented 9 years ago

Sounds good! I really wish there weren't these subtle discrepancies between GNU and BSD versions of these tools.

On 6 October 2014 23:12, Jeffrey Flanigan notifications@github.com wrote:

Thanks for tracking down what caused this error. I'm fixing this in the main branch in a slightly different way (using awk instead of head, which should be supported on OS X).

— Reply to this email directly or view it on GitHub https://github.com/jflanigan/jamr/pull/7#issuecomment-58141519.