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
my team has been implementing quite several utilities. some are close to core features, some other are more advanced and utilities. for example, their class names and features are like:
conservatively, i'd say these can be part of, say, torchdata-contrib. but i'd like to hear from the maintainers. where would you suggest drawing the line? any other suggestions would be great, too.
The text was updated successfully, but these errors were encountered:
Thanks @keunwoochoi for bringing this up! It is a very valid point of discussion. We had some discussions and this is one way we can go about it: let's keep all core node functionalities in torchdata/nodes (similar to Cycler, Header, etc that you are adding). For nodes that pertain to data access, lets include those which are general enough for wide use. For eg, CSV, JsonL, etc. We can discuss further to have them all in torchdata/nodes or torchdata/nodes/data. That said, we would be cautious of bringing in nodes that depend on exclusive company stacks or services like S3, GCP, etc, although we encourage users to continue building on existing nodes to unblock their specific needs.
Regarding round robin, we already have a PR open which @ramanishsingh is taking care of. Edit: CSV Reader WIP here (PR1474)
my team has been implementing quite several utilities. some are close to core features, some other are more advanced and utilities. for example, their class names and features are like:
and some more.
conservatively, i'd say these can be part of, say,
torchdata-contrib
. but i'd like to hear from the maintainers. where would you suggest drawing the line? any other suggestions would be great, too.The text was updated successfully, but these errors were encountered: