-
Notifications
You must be signed in to change notification settings - Fork 27
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
Issues loading pretrained model #21
Comments
Merged
In addition to the fix in #22, I think it might make sense to move the weights into the |
Yes I think that makes sense, although it needs to be tested if that would resolve the issue. Pathlike reference to package files doesn't always work on all systems. |
Okay - I'll make this change (with some tests) in addition to the notes you left in #20 to a new PR |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When I'm trying to install Presto as a package not in editable mode (e.g. to pack it to run on our cluster), I'm experiencing a couple of issues:
load_pretrained
method here but at the moment this is not possibledata
folder manually and usingupdate_data_dir
from here, but while thedata_dir
variable is successfully updated,default_model_path
from here is not, so it has actually no effect for loading the pretrained modelThe text was updated successfully, but these errors were encountered: