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

drop example extra-requirements #178

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

12rambau
Copy link
Contributor

@12rambau 12rambau commented Oct 2, 2024

It's exactly the same as the dataflow one so let's keep it simple and only provide a "dataflow" one.

It's exactly the same as the dataflow one so let's keep it simple and only provide a "dataflow" one.
@12rambau 12rambau marked this pull request as ready for review October 2, 2024 11:13
@jdbcode
Copy link
Member

jdbcode commented Oct 2, 2024

At this point it does seem redundant, but when/if we have more examples (that use e.g., pytorch, scipy, etc), it would be nice to be able to install all example deps with xee[examples] (if I understand correctly). Why not leave this here as a structure to add to?

@12rambau
Copy link
Contributor Author

12rambau commented Oct 3, 2024

I see you point, maybe let's leave this one on stale for now and see what can be done elswhere (I don't fully understand the different level of dependencies yet)

@jdbcode
Copy link
Member

jdbcode commented Oct 3, 2024

Sounds good. Will leave open for now as a reminder to keep dep organization in mind.

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

Successfully merging this pull request may close these issues.

2 participants