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

TrustyAI Service Client #1152

Closed
alexcreasy opened this issue Apr 19, 2023 · 3 comments
Closed

TrustyAI Service Client #1152

alexcreasy opened this issue Apr 19, 2023 · 3 comments
Labels
feature/model-serving Model Serving Feature priority/high Important issue that needs to be resolved asap. Releases should not have too many of these.

Comments

@alexcreasy
Copy link
Contributor

alexcreasy commented Apr 19, 2023

Goal

Create a rest client to simplify sending / receiving data to the TrustyAI service

Dependency issue

Itemized goals

  • Create a rest client for the TrustyAI service
  • Create type definitions for Trusty request / response data
  • Update existing phase 0 work to use new type definitions

Aspects continued elsewhere

No response

Mocks

No mocks, non-visual feature

Linked TrustyAI issues:

@alexcreasy
Copy link
Contributor Author

After reviewing the mocks contributed by UX, I've asked the TrustyAI team for an additional API endpoint to provide the full list of all active metric requests.

The trusty team have provided that here, I'm waiting for this to be merged so it can be integrated into the TrustyAI client for the dashboard.

trustyai-explainability/trustyai-explainability#181

@alexcreasy
Copy link
Contributor Author

I've asked the TrustyAI team to look into removing unnecessary type hints that are required when submitting scheduled metric requests to their service in order to simplify the code needed for the frontend:

Tracking: https://issues.redhat.com/browse/RHODS-8871

@alexcreasy
Copy link
Contributor Author

alexcreasy commented May 30, 2023

The TrustyAI team have raised a PR for RHODS-8871 here: trustyai-explainability/trustyai-explainability#186

Once this has been merged, I'll update the API facing code covered in this issue.

@alexcreasy alexcreasy mentioned this issue May 31, 2023
6 tasks
@alexcreasy alexcreasy moved this from In progress to Done in ODH Dashboard Planning Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/model-serving Model Serving Feature priority/high Important issue that needs to be resolved asap. Releases should not have too many of these.
Projects
No open projects
Archived in project
Development

No branches or pull requests

1 participant