Skip to content

fix: remove trailing os sep in local pretrained model path#42493

Draft
junjunjd wants to merge 1 commit intohuggingface:mainfrom
junjunjd:fix/remove-trailing-os-sep
Draft

fix: remove trailing os sep in local pretrained model path#42493
junjunjd wants to merge 1 commit intohuggingface:mainfrom
junjunjd:fix/remove-trailing-os-sep

Conversation

@junjunjd
Copy link
Copy Markdown

Remove trailing OS separator in local pretrained model paths. Fix issue #42490.

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 read the contributor guideline,
    Pull Request section?
  • Was this discussed/approved via a Github issue or the forum? Please add a link
    to it if that's the case.
  • Did you make sure to update the documentation with your changes? Here are the
    documentation guidelines, and
    here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@junjunjd junjunjd force-pushed the fix/remove-trailing-os-sep branch 4 times, most recently from 566cdb9 to d83a1da Compare November 30, 2025 01:26
@junjunjd junjunjd marked this pull request as draft December 1, 2025 06:28
@junjunjd junjunjd force-pushed the fix/remove-trailing-os-sep branch 2 times, most recently from 4c0c836 to d443a5c Compare December 1, 2025 07:38
@junjunjd junjunjd force-pushed the fix/remove-trailing-os-sep branch from d443a5c to d7eac05 Compare December 19, 2025 04:52
@Rocketknight1
Copy link
Copy Markdown
Member

I believe this is fixed now, according to the discussion in #42490?

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