fix: Fill NAN and NAT fields before sending them to the frontend #394
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
I encountered an issue when trying to load a CSV file, exported from Google Sheets, that contained NAT or NAN values.
Example output after been read by Ibis:
In this case it doesn't throw any error in the UI, the dialog only shows No data to display.
In the console we have the following error:
Error: SyntaxError: JSON.parse: unexpected character at line 1 column 1714 of the JSON data
My proposal is to apply a fillna on the first 50 rows before they are sent back to the UI, so that the JSON can be correctly read on the client side.
Please let me know if you disagree with this solution.
Have a nice day !