You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
On some systems like our cluster, where a Python wheel containing Presto is shipped and deployed, this does not work. I fixed it for now locally by catching the error without failing and the rest of the code (computing Presto encodings) worked without issue on the cluster. So it's something to look at.
The text was updated successfully, but these errors were encountered:
Which functions are you using from the Presto codebase?
I think it would make sense to split a lot of the functionality out into a (much simpler) pip-installable package, but my guess is that only two things (the Presto model itself, and the utils function to make data) are necessary.
As a note, we do have single_file_presto.py which is intended for easy integration into other applications (since it just requires a single file to be copied into another application), but this doesn't solve the requirements issues.
At the moment only construct_single_presto_input and Presto.load_pretrained from which we use the encoder. But we're only getting started ;-)
I'll take a look at the single file implementation!
Hi guys,
during a Presto import, this line always gets triggered:
presto/presto/utils.py
Line 22 in 182d590
On some systems like our cluster, where a Python wheel containing Presto is shipped and deployed, this does not work. I fixed it for now locally by catching the error without failing and the rest of the code (computing Presto encodings) worked without issue on the cluster. So it's something to look at.
The text was updated successfully, but these errors were encountered: