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

RHOAIENG-3245: add methods to execute and return queries from trustyai service #202

Merged

Conversation

christinaexyou
Copy link
Contributor

addresses RHOAIENG-3245 by making adding the following methods:

  • converts json files to pandas dfs and vice versa
  • uploads data to service and/or model
  • get metric request with the option to schedule for reoccurring intervals
  • retrieves metric data from prometheus for specific time interval

demo nb to follow shortly

@christinaexyou christinaexyou force-pushed the RHOAIENG-3245 branch 3 times, most recently from 626fb3f to 29b5244 Compare February 15, 2024 16:52
src/trustyai/utils/extras/metrics_service.py Show resolved Hide resolved
src/trustyai/utils/extras/metrics_service.py Show resolved Hide resolved
src/trustyai/utils/extras/metrics_service.py Outdated Show resolved Hide resolved
src/trustyai/utils/extras/metrics_service.py Outdated Show resolved Hide resolved
src/trustyai/utils/extras/metrics_service.py Outdated Show resolved Hide resolved
src/trustyai/utils/extras/metrics_service.py Outdated Show resolved Hide resolved
src/trustyai/utils/extras/metrics_service.py Outdated Show resolved Hide resolved
src/trustyai/utils/extras/metrics_service.py Show resolved Hide resolved
src/trustyai/utils/extras/metrics_service.py Show resolved Hide resolved
src/trustyai/utils/extras/metrics_service.py Outdated Show resolved Hide resolved
@christinaexyou christinaexyou force-pushed the RHOAIENG-3245 branch 7 times, most recently from ca1f060 to f160dba Compare February 28, 2024 20:27
@christinaexyou
Copy link
Contributor Author

changes made:

  • added separate dependency for server module. upated pyproject.tomll and pyproject.toml to reflect this change
  • added tests for df to json conversion and vice versa in tests/extras/test_metrics_service.py
  • updated metrics_service.py by enabling SSL verification by default, adding configurable timeout, and raising exceptions

pyproject.toml Outdated Show resolved Hide resolved
src/trustyai/utils/server/server.py Outdated Show resolved Hide resolved
src/trustyai/utils/server/server.py Outdated Show resolved Hide resolved
@christinaexyou
Copy link
Contributor Author

changes made:

  • replaced new_client_from_config to load_incluster_config()
  • renamed server/server.py to api/api.py
  • updated metrics_service.py, pyproject.toml and workflow.yml to reflect name change

@ruivieira ruivieira merged commit 295b423 into trustyai-explainability:main Mar 5, 2024
5 checks passed
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.

2 participants