Skip to content
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

Open
WillAyd opened this issue Dec 5, 2019 · 9 comments
Open

Post to conda #46

WillAyd opened this issue Dec 5, 2019 · 9 comments
Labels
enhancement New feature or request

Comments

@WillAyd
Copy link
Collaborator

WillAyd commented Dec 5, 2019

No description provided.

@WillAyd
Copy link
Collaborator Author

WillAyd commented Dec 16, 2019

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

@WillAyd WillAyd added the enhancement New feature or request label Dec 21, 2019
@WillAyd
Copy link
Collaborator Author

WillAyd commented Feb 2, 2024

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 TableName objects. If that were duck-typed and we removed that dependency a conda package is not out of the question

FYI @jorwoods

@craigbosco
Copy link

Would love to get this on Conda... makes life a lot easier. I appreciate your work!

@zach-hunt
Copy link

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!

@WillAyd
Copy link
Collaborator Author

WillAyd commented Sep 19, 2024

@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)

@zach-hunt
Copy link

@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!

@craigbosco
Copy link

I could take a stab at it. I'll check on tableauhyperapi and see if that's feasible before attempting to remove the dependency.

@WillAyd
Copy link
Collaborator Author

WillAyd commented Sep 20, 2024

@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

@WillAyd
Copy link
Collaborator Author

WillAyd commented Sep 20, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants