Skip to content

orch integration#27

Merged
guywaldman merged 18 commits intomainfrom
orch-integration
Jul 22, 2024
Merged

orch integration#27
guywaldman merged 18 commits intomainfrom
orch-integration

Conversation

@guywaldman
Copy link
Owner

@guywaldman guywaldman commented Jul 22, 2024

Detailed Changes

Note: This is a major PR with too many changes for one PR (some unrelated), which is not something I usually like to do, but oh well.

This PR adds integration with the orch crate, and removes a lot of the things I added to magic-cli into a separate crate that is solely responsible for language model execution and orchestration.
There are more things I intend to introduce from orch (such as model error correction) but they will be separated into a different PR.

Additional Changes

  • Generates a default configuration if it doesn't exist
  • Added --output-only to suggest which only prints the output, without logs or ability to revise or copy/execute the command
  • Added dependency on orch and orch_response (latter will be bundled with orch in the future)
  • Refactor to accomodate for the orch crate
  • SOSupport for a configurable config path
  • Basic integration tests
  • CI for "DNC" comments, to make sure that stuff that isn't intended to be pushed, is indeed pushed

@guywaldman guywaldman changed the title Orch integration orch integration Jul 22, 2024
@guywaldman guywaldman merged commit f9b053c into main Jul 22, 2024
@guywaldman guywaldman deleted the orch-integration branch July 22, 2024 04:38
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.

1 participant