Skip to content

Endpoint improvements#59

Merged
sinnec merged 2 commits intomainfrom
api-fixes
Dec 14, 2022
Merged

Endpoint improvements#59
sinnec merged 2 commits intomainfrom
api-fixes

Conversation

@sinnec
Copy link
Copy Markdown
Contributor

@sinnec sinnec commented Dec 13, 2022

  • Fixed inference row xai report endpoint
  • Test for inference row xai report endpoint
  • Fixed some paths to use query params
  • Removed user API endpoints (get and update)
  • API documentation
  • Some refactoring

- Fixed inference row xai report endpoint
- Test for inference row xai report endpoint
- Fixed some paths to use query params
- Removed user API endpoints (get and update)
- API documentation
- Some refactoring
@sinnec sinnec requested a review from gcharis December 13, 2022 12:56
@sinnec sinnec changed the title Endpoint improvements: Endpoint improvements Dec 13, 2022
@sinnec sinnec requested a review from momegas December 13, 2022 13:49
Comment thread src/api/v1/inference_rows.py Outdated
Copy link
Copy Markdown
Contributor

@gcharis gcharis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sinnec sinnec merged commit 7c01e63 into main Dec 14, 2022
@gcharis gcharis deleted the api-fixes branch December 20, 2022 12:17
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

Successfully merging this pull request may close these issues.

3 participants