Skip to content

Adding a flag whether to save checkpoint or not in run_lora_clm.py#736

Merged
regisss merged 2 commits into
mainfrom
save_last_ckpt
Feb 27, 2024
Merged

Adding a flag whether to save checkpoint or not in run_lora_clm.py#736
regisss merged 2 commits into
mainfrom
save_last_ckpt

Conversation

@yeonsily
Copy link
Copy Markdown
Collaborator

This is the same change as PR #711 but for different script.

What does this PR do?

Fixes # (issue)

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@yeonsily yeonsily requested a review from libinta February 23, 2024 23:09
@yeonsily yeonsily requested a review from regisss as a code owner February 23, 2024 23:09
@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@regisss
Copy link
Copy Markdown
Collaborator

regisss commented Feb 25, 2024

@yeonsily Can you run make style please?

@yeonsily
Copy link
Copy Markdown
Collaborator Author

@yeonsily Can you run make style please?

Strange.. I didn't see any style change from make style.. Let me check further.

Copy link
Copy Markdown
Collaborator

@regisss regisss left a comment

Choose a reason for hiding this comment

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

LGTM

@yeonsily I've seen a few occurrences where the same version of ruff produces different results in two different environments. It's rare enough so that it's not really an issue but it may be the reason why you got a different output 🙁

@regisss regisss merged commit d7291dd into main Feb 27, 2024
@regisss regisss deleted the save_last_ckpt branch February 27, 2024 01:35
jychen21 pushed a commit to jychen21/optimum-habana that referenced this pull request Feb 27, 2024
puneeshkhanna pushed a commit to puneeshkhanna/optimum-habana-fork that referenced this pull request Mar 11, 2024
HolyFalafel pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Mar 11, 2024
gplutop7 pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Oct 15, 2025
…gface#736)

Co-authored-by: Grzegorz Pluto-Prondzinski <gplutopx@habana.ai>
gplutop7 pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Nov 6, 2025
…gface#736)

Co-authored-by: Grzegorz Pluto-Prondzinski <gplutopx@habana.ai>
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.

3 participants