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

OPIK-556: Add embedded python service POC #955

Merged
merged 2 commits into from
Dec 24, 2024

Conversation

andrescrz
Copy link
Collaborator

@andrescrz andrescrz commented Dec 23, 2024

Details

This is a proof of concept of a Python service, based on Flask, capable of running user defined Python code containing a BaseMetric definition and returning a ScoreResult (or a list of them).

The steps to formalise this PoC as an actual implementation will be documented as stories under epic OPIK-588. Some of those are:

  • Embed the service inside Opik backend.
  • Define the data mapping interface.
  • Sandbox and secure the service.
  • Productionalize the service.
  • Add CI pipeline.

Issues

Resolves OPIK-556

Testing

  • Manually tested.
  • Created unit tests.
  • CI pipeline for this service not in place yet.

Documentation

@andrescrz andrescrz self-assigned this Dec 23, 2024
@andrescrz andrescrz added the enhancement New feature or request label Dec 23, 2024
@andrescrz andrescrz force-pushed the andrescrz/OPIK-566-add-python-embedded-service-poc branch from 8b8006e to ccdd0f3 Compare December 23, 2024 16:31
@andrescrz andrescrz changed the title OPIK-566: Add embedded python service POC OPIK-556: Add embedded python service POC Dec 23, 2024
@andrescrz andrescrz marked this pull request as ready for review December 23, 2024 16:32
@andrescrz andrescrz requested review from a team as code owners December 23, 2024 16:32
Nimrod007
Nimrod007 previously approved these changes Dec 24, 2024
@andrescrz andrescrz merged commit f8d6678 into main Dec 24, 2024
@andrescrz andrescrz deleted the andrescrz/OPIK-566-add-python-embedded-service-poc branch December 24, 2024 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants