Merged
Conversation
…2960) * adding util to truncate on tokens * using token truncator before summarizing * switch to langchain chatlitellm * fixing test * fix tests * docstring * adding test * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * reverting temperature change * moving to top-level drop params * factor in output tokens * fixing output calc for flashcards * fix tests * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* chore: migrate from poetry/pip to uv for dependency management - Run migrate-to-uv to convert pyproject.toml and generate uv.lock - Update Dockerfile to install uv and use uv sync instead of poetry - Update GitHub Actions CI to use astral-sh/setup-uv and uv run/sync Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: use uv run in test scripts Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: pin Python 3.12 in CI to match Dockerfile Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: override setuptools<80 to fix pkg_resources import error mitol-django-scim uses pkg_resources.declare_namespace() which was removed from setuptools>=80. llama-index-core requires setuptools>=80.9.0 as a declared dep but doesn't actually use it at runtime, so override-dependencies is safe here. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: pin youtube-transcript-api<1.1 and suppress jwt InsecureKeyLengthWarning youtube-transcript-api 1.1+ removed the class method list_transcripts in favor of an instance method API. Pin to 1.0.x to match original behavior. PyJWT 2.11+ warns when HMAC key is < 32 bytes. Tests intentionally use short keys, so suppress jwt module UserWarnings in the test fixture. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: replace poetry run with uv run in AGENTS.md Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: pre-create /opt/venv with correct ownership before switching to mitodl user uv creates the venv at UV_PROJECT_ENVIRONMENT (/opt/venv) on first sync. Running uv sync as the mitodl user fails because /opt/venv doesn't exist and the non-root user cannot create it. Pre-create it with chown. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * ci: update setup-uv to v7, enable caching, use uv sync --locked Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
odlbot
added
deploying to rc
waiting for checkboxes
all checkboxes checked
and removed
deploying to rc
waiting for checkboxes
labels
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tobias Macey
Shankar Ambady