-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Euros a problem #7
Comments
Oh. I think I know why. The values are all denoted with commas for decimals. "€26,56" Can you fix it or should I just regex the files before using the tool? |
Nope - that did not fix it. ValueError: could not convert string to float: '€26.56' Must be the euro symbol maybe, but looking at the code, there appears to be currency matching going on so I don't think I can just change it to a dollar symbol or remove it without impacting something else. |
# [v1.0.5](v1.0.4...v1.0.5) (2024-12-11) ## 🐛 Bug Fixes - [`d2eab76`](d2eab76) euros formatting (Issues: [`#7`](#7)) [skip ci]
🎉 This issue has been resolved in version 1.0.5 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Let me know if this didn't resolve the issue 🤝 lunchable-primelunch/lunchable_primelunch/primelunch.py Lines 220 to 222 in d2eab76
|
@andersonimes could you share a CSV with a few test records so I can debug on my end? |
I hope I don't regret this :) |
Hello! Did you get a chance to take a look at this? |
Yup, thanks for sending that! This was a tough fix but I believe I've got it. If you run into any issues try setting the |
Let me know how the new version works 😅 |
🎉 This issue has been resolved in version 1.0.6 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
@juftin sorry to say I think you should reopen this. |
Hello! I am getting the following error trying to process a file. I'm wondering if maybe this isn't designed to process a file with Euros, maybe? Here is the error:
The text was updated successfully, but these errors were encountered: