-
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
Post to conda #46
Comments
I think will need for the tableauhyperapi to at least be available on pip for this, or else the install scripts be pretty tough to maintain |
Now that we are bundling the Hyper API this is a little closer to reality. However, we still have a runtime dependency on the tableauhyperapi with the sole purpose of accepting FYI @jorwoods |
Would love to get this on Conda... makes life a lot easier. I appreciate your work! |
Seconded! Having a bit of a go getting pip to install things nicely with the rest of the environment, so conda would be quite a treat. Thanks! |
@craigbosco @zach-hunt would either of you be interested in contributing and maintaining that recipe? From my loose understanding, I think we need something as described here: https://conda-forge.org/docs/maintainer/adding_pkgs/ The one blocker right now is that we have a dependency on the tableauhyperapi. However, we could either try to create that on conda-forge (with permission from Tableau) or drop that runtime dependency (requires some code updates) |
@WillAyd - I don't have any experience with publishing to conda, so I wouldn't be the guy to support this. I do hope this project continues to succeed! |
I could take a stab at it. I'll check on tableauhyperapi and see if that's feasible before attempting to remove the dependency. |
@craigbosco that's great. To help that front I've started to remove the runtime dependency on the tableau hyper API in #336. That is desirable for more than just packaging as it reduces the runtime installation size of pantab by somewhere between 50-100 MB |
OK I've dropped the dependency on the tableauhyperapi - @craigbosco I think you can just set up a conda recipe using this repo as is. If you get stuck while looking at the setup feel free to ping - happy to offer any guidance that I can |
No description provided.
The text was updated successfully, but these errors were encountered: