Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix it to use env vars if available #1063

Merged
merged 3 commits into from
Feb 4, 2025
Merged

Conversation

michaelneale
Copy link
Collaborator

this will ensure that GOOSE_PROVIDER and GOOSE_MODEL are used vs hard coded values

@@ -60,7 +60,7 @@ fn setup_profile(temp_dir_path: &Path, profile_string: Option<&str>) {
"profile_items": {
"default": {
"provider": "databricks",
"model": "claude-3-5-sonnet-2",
"model": "goose",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"claude-3-5-sonnet-2" is not a generic name, so may as well make it use an alias as that is what it is in databricks anyway

@michaelneale michaelneale merged commit dd55c42 into main Feb 4, 2025
4 checks passed
@michaelneale michaelneale deleted the micn/urgent-model-fix branch February 4, 2025 08:11
michaelneale added a commit that referenced this pull request Feb 6, 2025
* main:
  Respect the terminal emulator's base colors (#1098)
  fix: add fallback sans-serif font to desktop app (#982)
  feat: add bedrock provider (#1069)
  feat: removed minHeight (#938)
  fix: add logs for goose run (#1085)
  docs: add langfuse observability guide (#1047)
  docs: updated bluesky link (#1078)
  docs: jetbrains tutorial (#1074)
  docs: add simple docs link (#1077)
  fix: apply cache control correctly to tools (#1071)
  fix: show default value when adding new one (#1057)
  fix it to use env vars if available (#1063)
  docs site: add PR previews (#865)
  chore: Better typing for preload.ts (#1055)
  fix: Allow port 443 to be used with https (ollama provider) (#1052)
laanak08 pushed a commit that referenced this pull request Feb 13, 2025
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.

2 participants