-
Notifications
You must be signed in to change notification settings - Fork 42
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
Quick bump - Snowpark_venv issues with frame_to_hyper #348
Comments
Thank you - I did overlook that message. Hmm this is a tough one...my guess is that it is an issue just on Windows (?). If you have the capability of running from WSL you might not run into the issue there (?) There are a few macros for Windows that maybe can be re-evaluated more closely. If you take out all of the string data does it then work? The hard part for me as a maintainer of pantab is that I don't have access to a Windows machine, and am not very familiar with Windows tools for debugging. Additionally, the Hyper API makes it impossible to run ASAN/UBSAN to detect any potential code issues in an automated fashion...I've brought that up with the Tableau team in the past but I don't know that there will be that much traction on that request, unfortunately |
I am creating process improvements for a BI team, and they ultimately need to be the ones to use my work with minimum additional set up / support... WSL wouldn't be an option for them I'm getting the same jupyter log with this:
I appreciate all your work! Sorry for the challenges, that sounds frustrating |
The jupyter log is unfortunately not helpful for this; it appears that there is a lower level failure that won't be captured in those logs. You are getting this when installing the library from github, but not from pypi right? Its possible that the CMake setup may be missing some corrections to the shared libraries that cibuildwheel and delvewheel take care of when distributing Windows binaries. Since the code in your most recent example has not been affected by any changes since the last release on pypi, it may be resolved when I post the new release (assuming the current release is working fine) |
You are probably right... yes, this was off of the Github branch. I uninstalled, and reinstalled from pip, and it isn't crashing anymore- but I am back to my Sort of a catch-22 until I can install the string fixes from pip |
What might be nice is if we set up nightly or release candidate builds of pantab to help with this. That is not something I personally have the bandwidth to take on right now, but @jorwoods has done a lot of great stuff for our CI so maybe has time/interest. Otherwise we would need a community contribution to make this happen! |
@skyth540 actually I was able to get release candidates distributed without too much extra trouble. Can you try python -m pip install --upgrade --pre pantab to get the 5.1.0 release candidate and see if it fixes your issues? |
Everything was successful up until frame_to_hyper
I have strings, categoricals, f32, f64, int8, int64, and date datatypes. |
The issue is that you are asking for a If you need to use database version 1 for your other tools, you will have to cast your 32 bit floats to 64 bit to use that version. For more info, see the tableau documentation on that parameter: https://tableau.github.io/hyper-db/docs/hyper-api/hyper_process/#default_database_version |
Let me know if you run into any other issues - hoping to cut a release next week if all else is good. Especially since Python 3.13 just came out would be nice to get a new release with wheels for that |
Going to go ahead and close this but if anything else comes up let me know |
Bumping this comment that may have been missed
Running this code:
Results in this error:
The text was updated successfully, but these errors were encountered: