-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Pretrained.from_hparams() failing on windows when non admin #1155
Comments
EDIT: it works well for following configuration as a workround, I let you decide if you need a better solution :) EncoderClassifier.from_hparams(
source=r"C:\path\to\pretrained\model",
savedir=r"C:\path\to\pretrained\model",
overrides={"pretrained_path": r"C:\path\to\pretrained\model"}) |
Yes, this is a known issue, see for instance #893. Thanks for a new workaround idea! At the moment we don't plan to change this implementation just for Windows users who cannot run with the right permissions. However, this might get solved as a side product if we decide to move away from symlinks to real copies - which seems to be behaviour that multiple users expect. |
I found a new one with latest version 0.5.11: A fetch of a |
Ooh that is just a bug in how local |
FYI this issue still exists for Tacotron2.from_hparams on Windows. |
Hello, Any news on this issue please? Thanks. |
I'd also be interested in an update. There are quite a few of us Windows users out there without admin rights 😅 Even some further explanation / documentation of the workarounds would be useful, although in my case I'm using a package that depends on speechbrain so I can't implement any workarounds directly. |
The following code is failing on windows with non-admin rights (not a big issue but sadly a "standard" case in lots of "big" companies) due to symlinks.
I fixed it locally by copying models to
savedir
instead of making a symlink (fetching.py:L102
), BUT I will not submit it as a fix.I can start working on local loading feature in a PR if you're interested. When loading from local, we could even avoid copying or symlinking directories and loading directly from the local folder for example.
The text was updated successfully, but these errors were encountered: