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

Support bot #378

Merged
merged 99 commits into from
Aug 31, 2023
Merged

Support bot #378

merged 99 commits into from
Aug 31, 2023

Conversation

KuuCi
Copy link
Contributor

@KuuCi KuuCi commented Jun 21, 2023

Completed incorporation of MosaicML model and embedding over OpenAI.

@KuuCi KuuCi enabled auto-merge (squash) June 21, 2023 00:17
Copy link
Collaborator

@dakinggg dakinggg left a comment

Choose a reason for hiding this comment

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

A few high level comments/things to fix in multiple places:

  • avoid hardcoded paths/names everywhere (file paths in yamls, endpoints, cluster names, etc)
  • please do a pass through the readme and make sure it explains the flow that you want it to. Right now it seems like the code and yaml are written for the 30b but the readme is written for 7b, and there are still a handful of references to the sec 10k example
  • all installs of foundry/composer should be pinned to specific commits
  • you'll need to run precommit for formatting and type checking, and add your folder to linting (
    - end-to-end-examples/sec_10k_qa
    ) so that it runs in CI

@dakinggg dakinggg disabled auto-merge August 31, 2023 07:28
Copy link
Collaborator

@dakinggg dakinggg left a comment

Choose a reason for hiding this comment

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

Please update the PR description to more accurately describe what is in this PR. LGTM!

@KuuCi KuuCi merged commit 686e73c into mosaicml:main Aug 31, 2023
22 checks passed
@KuuCi KuuCi deleted the support-bot branch August 31, 2023 23:21
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