Skip to content
This repository was archived by the owner on Apr 23, 2026. It is now read-only.

Linux test - #1314

Closed
makelinux wants to merge 2 commits into
instructlab:mainfrom
RedHat-Israel:linux_test
Closed

Linux test#1314
makelinux wants to merge 2 commits into
instructlab:mainfrom
RedHat-Israel:linux_test

Conversation

@makelinux

@makelinux makelinux commented Jun 9, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@mergify mergify Bot added the ci-failure PR has at least one CI failure label Jun 9, 2024
Refactoring for linux_test.

Signed-off-by: Costa Shulyupin <costa.shul@redhat.com>
@mergify mergify Bot added ci-failure PR has at least one CI failure and removed ci-failure PR has at least one CI failure labels Jun 9, 2024
Almost all code of linux_test() is moved from linux_train().
The only new line is

  model = AutoModelForCausalLM.from_pretrained(merged_model_dir)

which loads previously saved model at:

  model.save_pretrained(merged_model_dir)

Signed-off-by: Costa Shulyupin <costa.shul@redhat.com>
@mergify mergify Bot removed the ci-failure PR has at least one CI failure label Jun 10, 2024
@leseb leseb added the hold In-progress PR. Tag should be removed before merge. label Jun 10, 2024
@leseb

leseb commented Jun 10, 2024

Copy link
Copy Markdown
Contributor

Putting on hold since more testing is required. @makelinux let us know when this is ready, thanks!

@makelinux
makelinux marked this pull request as draft June 11, 2024 09:37
@mergify mergify Bot added the needs-rebase This Pull Request needs to be rebased label Jun 11, 2024
@mergify

mergify Bot commented Jun 11, 2024

Copy link
Copy Markdown
Contributor

This pull request has merge conflicts that must be resolved before it can be
merged. @makelinux please rebase it. https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@nathan-weinberg nathan-weinberg mentioned this pull request Jun 12, 2024
2 tasks
@makelinux makelinux closed this Jun 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

hold In-progress PR. Tag should be removed before merge. needs-rebase This Pull Request needs to be rebased

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants