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

Wanted, more information when I try to upload a CSV containing an error #1527

Open
DorinevanderVlies opened this issue Oct 26, 2016 · 2 comments
Assignees

Comments

@DorinevanderVlies
Copy link
Contributor

Currently when a CSV containing one or more errors is saved the following error message pops up:

image

This makes debugging the CSV pretty difficult. It would be very nice if some more feedback about the cause of the error was provided.

@grdw
Copy link
Contributor

grdw commented Oct 27, 2016

I'm thinking this is part of a larger problem. In a way I understand why we support CSV editing in another way I would find it way better if we could get rid of it. 😈 Because you use the CSV editor it means that there's something wrong with the profile matrix editor or some essential feature(s) missing. I would like to know why it's preferable to use the CSV as input over the profile matrix editor?

I know that the CSV; for instance; mallforms Floats and Integers to Strings which causes problems like the one's were fixing in #1517. Typo's are more easily made and unfortunately this issue might take up more time than it should. It could be that the CSV is parsed just fine but the underlying calculation doesn't know what to do with the input due to missing numbers, missing attributes, accidentally deleted rows, but most likely - and this is purely judging by the Airbrake errors of late - Floats and Integers (as in numbers) that are interpreted as text.

Also in line with current ideas to merge the Topology and the technology matrix together into a single set of data, the CSV might be of little use or might become to much of a burden to keep supporting.

These are my 2 cents on the issue.

Ping @ChaelKruip @antw

@grdw grdw modified the milestone: Replacing CSV import/export Nov 1, 2016
@grdw
Copy link
Contributor

grdw commented Nov 1, 2016

@DorinevanderVlies could you maybe send me a CSV that doesn't currently work on ETmoses?

@grdw grdw removed their assignment Jan 5, 2018
@antw antw removed their assignment Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants