feat: add --switch-config-path parameter to llm-transpile command#2256
feat: add --switch-config-path parameter to llm-transpile command#2256hiroyukinakazato-db wants to merge 8 commits intomainfrom
Conversation
Add support for specifying a custom Switch configuration file when running LLM transpilation. The path must be a workspace path starting with /Workspace/. When not specified, Switch uses its default config. Closes #2255
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2256 +/- ##
==========================================
+ Coverage 66.41% 66.43% +0.02%
==========================================
Files 99 99
Lines 9094 9100 +6
Branches 974 977 +3
==========================================
+ Hits 6040 6046 +6
Misses 2878 2878
Partials 176 176 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
✅ 145/145 passed, 8 flaky, 4 skipped, 47m12s total Flaky tests:
Running from acceptance #4014 |
E2E Test EvidenceTest 1: Without
|
gueniai
left a comment
There was a problem hiding this comment.
We need details to the docs as well
|
Documentation updated. @gueniai @asnare @sundarshankar89 Ready for review! |
sundarshankar89
left a comment
There was a problem hiding this comment.
Before I approve this, can we think about some plan or effort on how to manage these prompts throught mlflow prompt registry. Can we come up with an effort and plan for that activity before exposing these flags to users.
Summary
--switch-config-pathparameter tollm-transpileCLI command/Workspace/(validation included)Closes #2255
Test plan
--switch-config-pathworks with actual workspace executionswitch_config_pathjob parameter