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

reckon fails any csv with Date::Error #121

Closed
mikaeljagelid opened this issue Apr 10, 2023 · 3 comments
Closed

reckon fails any csv with Date::Error #121

mikaeljagelid opened this issue Apr 10, 2023 · 3 comments

Comments

@mikaeljagelid
Copy link

I'm just trying out reckon for beancount but can't get it to parse any csv I try. I always get the error:

/Users/.../.gem/ruby/2.6.0/gems/reckon-0.9.1/lib/reckon/date_column.rb:76:in `rescue in likelihood': uninitialized constant Date::Error (NameError)

I've tried csv from my banks and example csv files for beancount-import with arguments for skipping header rows, the correct date column, encoding, etc but every file fails in the same way.

@benprew
Copy link
Collaborator

benprew commented Apr 13, 2023

Hi @mikaeljagelid thanks for reporting this error. I'm testing out a fix now, I'll update this issue when it's available for testing.

@benprew
Copy link
Collaborator

benprew commented Apr 14, 2023

@mikaeljagelid This appears to be a ruby 2.6.0 issue, it doesn't occur on 2.7+.

I'm currently testing on 2.7, 3.0 and 3.1. For my curiosity, what OS are you running? I'll add 2.6.0 to the list of rubies to test, but I'd like to know where that's coming from.

Thanks!

@benprew
Copy link
Collaborator

benprew commented Apr 14, 2023

@mikaeljagelid , I've released Reckon 0.9.2 which should fix this error /~https://github.com/cantino/reckon/releases/tag/v0.9.2

Give it a try and if it doesn't fix it, reopen this issue.

Thanks!

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