fix(jig): don't set base url if it's the same as default#263
Open
technillogue wants to merge 8 commits intonextfrom
Open
fix(jig): don't set base url if it's the same as default#263technillogue wants to merge 8 commits intonextfrom
technillogue wants to merge 8 commits intonextfrom
Conversation
* Update README.md (#245) * Add from_checkpoint parameter to price estimation for FT Job creation (#247) * codegen metadata * codegen metadata * codegen metadata * jig papercuts (#238) * feat(jig): show a unique name tip when deployment create fails * feat(jig): factor out config_path option and add short flag * feat: jig support for multi deployment * codegen metadata * chore: Update descriptions for jig queue methods and properties * chore(internal): bump dependencies * Allow tool calls through together-py and remove alternating roles check (#244) * Update README.md (#245) * Add from_checkpoint parameter to price estimation for FT Job creation (#247) * codegen metadata * codegen metadata * codegen metadata * Allow tool calls through together-py and remove outdated checks * Remove outdated tests * lint * Enforce that either content or tool calls exists * Skip for empty content * Condition field checks on role --------- Co-authored-by: Blaine Kasten <blainekasten@gmail.com> Co-authored-by: Artem Chumachenko <artek.chumak@gmail.com> Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com> * make submit --watch signal status through exit code * fix(cli): fine-tuning retrieve now renders data instead of schema (#250) * fix(jig): lint errors * fix(jig): migrate old state files properly and be even more defensive about parsing deploy errors * fix(jig): pyright does not handle isinstance type narrowing in ternary expressions. also fix migration logic * format code * Feat: Allow uploading local file during fine tuning create call in the CLI * cleanup * Add support for validation_file * fix lint --------- Co-authored-by: Artem Chumachenko <artek.chumak@gmail.com> Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com> Co-authored-by: technillogue <technillogue@gmail.com> Co-authored-by: Mohamad Rostami <mbrostami@proton.me> Co-authored-by: Conner Manuel <57027354+connermanuel@users.noreply.github.com>
* feat(jig): Format jig status - chore(jig): typing fix - chore(jig): minor improvements to status - fix(jig): status formatter use Typed models - fix(jig): reformat app status - fix(jig): wip: reformat config status - fix(jig): add env to config status - fix(jig): replica event status - fix(jig): remove unused age func - fix(jig): simplify and condense status message - fix(jig): only show 'ready since' for running replicas - fix: fixed replica event grouping and image tag split - fix: Fix minor status errors - fix(jig): show replica_id instead of revision_id for replica events in jig status * minor mypy type error * chore: format_deployment_status(deployment) -> format_deployment_status(d) * fix(jig): profile -> metric, targetValue -> target --------- Co-authored-by: technillogue <technillogue@gmail.com>
blainekasten
changed the title
| env_vars = [{"name": k, "value": v} for k, v in config.deploy.environment_variables.items()] | ||
| env_vars.append({"name": "TOGETHER_API_BASE_URL", "value": _get_api_base_url(client)}) | ||
|
|
||
| if (cli_base_url := _get_api_base_url(client)) != "https://api.together.ai": |
Contributor
There was a problem hiding this comment.
fwiw - pretty sure the base client is always defaulting to api.together.xyz currently
Contributor
There was a problem hiding this comment.
I think we have this string declared in a constants file somewhere
stainless-app
bot
force-pushed
the
next
branch
from
5bcfbdf to
bfb1918
Compare
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
4 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.
Uh oh!
There was an error while loading. Please reload this page.