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
Fix serialization of numpy arrays and pandas dataframes in REST API #2838
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow, elegant fix! Good to go 👍
Unfortunately this breaks the feedback get. So even if there's a solution to the"PR stuck" issue I have to fix that before merge. |
We had no tests for it? Stuff like this makes me wish we had a coverage report somewhere 😕 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add tests for the GET /feedback
endpoint first
@ZanSara I added some tests for the search API regarding non primitive types. Feedback API had already some tests. However, the mock never returned any labels and the assertions of the response objects were missing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, left only a comment for style
Related Issue(s):
Proposed changes:
json_encoders
for numpy arrays and pandas Dataframes