Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Apr 30, 2025

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

Ensure that the latest version for instructlab-training is consumed: https://github.com/instructlab/training/releases/tag/v0.10.1


This is an automatic backport of pull request #3312 done by Mergify.

jaideepr97 and others added 3 commits April 23, 2025 03:15
Signed-off-by: Jaideep Rao <[email protected]>
(cherry picked from commit 8ea9979)
missed adding logic to pick up teacher model id from config in earlier PR. This PR fixes that 







**Checklist:**

- [ ] **Commit Message Formatting**: Commit titles and messages follow guidelines in the
  [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/#summary).
- [ ] [Changelog](https://github.com/instructlab/instructlab/blob/main/CHANGELOG.md) updated with breaking and/or notable changes for the next minor release.
- [ ] Documentation has been updated, if necessary.
- [ ] Unit tests have been added, if necessary.
- [ ] Functional tests have been added, if necessary.
- [ ] E2E Workflow tests have been added, if necessary.
<hr>This is an automatic backport of pull request #3309 done by [Mergify](https://mergify.com).


Approved-by: courtneypacheco

Approved-by: jaideepr97
Ensure that the latest version for `instructlab-training` is consumed: https://github.com/instructlab/training/releases/tag/v0.10.1

Signed-off-by: Courtney Pacheco <[email protected]>
Signed-off-by: Charlie Doern <[email protected]>
(cherry picked from commit 5b56fca)
@mergify mergify bot added release-branch Pull Request directly to a release branch one-approval PR has one approval from a maintainer dependencies Relates to dependencies labels Apr 30, 2025
@mergify mergify bot added ci-failure PR has at least one CI failure and removed one-approval PR has one approval from a maintainer labels Apr 30, 2025
@ktdreyer ktdreyer added this to the 0.26.0 milestone Apr 30, 2025
@booxter
Copy link
Contributor

booxter commented Apr 30, 2025

We need to land #3329 first to pass unit tests.

@cdoern
Copy link
Contributor

cdoern commented Apr 30, 2025

closing because of #3334

@cdoern cdoern closed this Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-failure PR has at least one CI failure dependencies Relates to dependencies release-branch Pull Request directly to a release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants