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

TruLens 1.2.9 #1671

Merged
merged 10 commits into from
Dec 4, 2024
Merged

TruLens 1.2.9 #1671

merged 10 commits into from
Dec 4, 2024

Conversation

sfc-gh-chu
Copy link
Contributor

@sfc-gh-chu sfc-gh-chu commented Nov 26, 2024

Description

Memoize base endpoint creation for cost tracking @sfc-gh-chu

add importlib resources to conda build @sfc-gh-chu

Switch from SQL function to REST API backend for Cortex Complete - co… @sfc-gh-dhuang

fix meta.yaml spacing @sfc-gh-chu

Fix Cortex provider for tests. @sfc-gh-dkurokawa

relax snowflake-ml-python version @sfc-gh-chu

Fix encoding issues in dashboard @sfc-gh-chu

Other details good to know for developers

Please include any other details of this change useful for TruLens developers.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)
  • New Tests
  • This change includes re-generated golden test results
  • This change requires a documentation update

Important

Update to version 1.2.9 with Snowflake and Cortex integration improvements, new Jenkinsfile for CI/CD, and test setup enhancements.

  • Version Update:
    • Update version to 1.2.9 in pyproject.toml and meta.yaml across multiple modules.
  • Snowflake and Cortex Integration:
    • Replace Snowflake connection with snowpark_session in Cortex class in provider.py.
    • Update test_endpoints.py and test_snowflake_feedback_evaluation.py to use snowpark_session.
    • Set Cortex.DEFAULT_SNOWPARK_SESSION in SnowflakeTestCase in snowflake_test_case.py.
  • Testing and CI/CD:
    • Add Jenkinsfile for CI/CD pipeline with Docker setup and test execution.
    • Add test-snowflake target in Makefile for Snowflake-specific tests.
  • Dependency Changes:
    • Add snowflake-ml-python to requirements.optional.txt and server_side_evaluation_artifacts.py.
    • Comment out dill dependency in src/core/pyproject.toml.
  • Miscellaneous:
    • Update meta.yaml files for version consistency.
    • Minor code refactoring and cleanup in app.py and endpoint.py.

This description was created by Ellipsis for 0614871. It will automatically update as commits are pushed.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Nov 26, 2024
@dosubot dosubot bot added the dependencies Pull requests that update a dependency file label Nov 26, 2024
@sfc-gh-chu sfc-gh-chu changed the base branch from main to releases/rc-trulens-1.2.8 November 26, 2024 23:55
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Nov 26, 2024
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Contributor

@sfc-gh-dhuang sfc-gh-dhuang left a comment

Choose a reason for hiding this comment

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

thanks for dilling with this

@sfc-gh-jreini
Copy link
Contributor

@coreyhu can you merge and create a github release for this?

@sfc-gh-chu sfc-gh-chu changed the base branch from releases/rc-trulens-1.2.8 to main December 2, 2024 18:27
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Dec 2, 2024
@sfc-gh-chu sfc-gh-chu changed the base branch from main to releases/rc-trulens-1.2.8 December 3, 2024 00:43
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Dec 3, 2024
@sfc-gh-chu sfc-gh-chu changed the base branch from releases/rc-trulens-1.2.8 to main December 3, 2024 23:47
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Dec 3, 2024
@sfc-gh-chu sfc-gh-chu merged commit 4827e97 into main Dec 4, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants