Complete LM/Backend Interoperability #674
Merged
Conversation
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
…r-to-completed-version Update skycamp2023.ipynb
Improvements to Signature
* Add Ruff linting workflow * Update GitHub Actions workflows * Make ruff apply but not fail * Style fixes by Ruff * Combine Workflows * Add caching * Add import related rules * Automatic Style fixes * Try and fix caching * Fix import * Use natve python dep caching * Update actions/checkout to v4 * Use autofix in tests * Try to cache "install poetry" * Empty-Commit to test caching * Remove workflow deps * Rename action * Update workflow name * Fix Docker stop command and subprocess run check * Fix functional imports --------- Co-authored-by: isaacbmiller <isaacbmiller@users.noreply.github.com>
* Add Ruff linting workflow * Update GitHub Actions workflows * Make ruff apply but not fail * Style fixes by Ruff * Combine Workflows * Add caching * Add import related rules * Automatic Style fixes * Try and fix caching * Fix import * Use natve python dep caching * Update actions/checkout to v4 * Use autofix in tests * Try to cache "install poetry" * Empty-Commit to test caching * Remove workflow deps * Rename action * Update workflow name * Fix Docker stop command and subprocess run check * Fix functional imports * Remove --no-root flag --------- Co-authored-by: isaacbmiller <isaacbmiller@users.noreply.github.com>
A few more typed examples
Formatting for new optimizers
…tor-amir # Conflicts: # dsp/primitives/predict.py # dsp/utils/settings.py # dspy/predict/predict.py # dspy/primitives/prediction.py # dspy/signatures/signature.py # dspy/teleprompt/bootstrap.py # dspy/teleprompt/signature_opt.py # dspy/teleprompt/signature_opt_bayesian.py # dspy/utils/dummies.py # poetry.lock # pyproject.toml # tests/functional/test_functional.py # tests/predict/test_predict.py # tests/primitives/test_program.py # tests/teleprompt/test_copro_optimizer.py # tests/teleprompt/test_mipro_optimizer.py
…en lm and input arguments are accomodated for
thomasahle
reviewed
Mar 19, 2024
Collaborator
thomasahle
left a comment
thomasahle
left a comment
There was a problem hiding this comment.
Just some random comments from sampling all the files
| import regex | ||
|
|
||
| from dspy.primitives.example import Example | ||
| from dspy.signatures.signature import Signature |
Collaborator
There was a problem hiding this comment.
We can just do from dspy import Signature
| from dspy.signatures.signature import ensure_signature | ||
|
|
||
| import pydantic | ||
| import ujson |
Collaborator
There was a problem hiding this comment.
This all seems to be stuff that's already in main...
7 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
27 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.
This PR completes the Backend/Language Model v1 interoperability feature for DSPy.
All tests are passing, and we should be fairly close to merge to main.
There were a few small changes to COPRO/MIPRO optimizers, that I believe are due to inconsistencies in the functionality, so we may want to double check those.
@CyrusOfEden