-
Notifications
You must be signed in to change notification settings - Fork 239
Move Experimental to TorchAO prototype #1899
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
Comments
I'd argue for the stronger claim of promoting the work out of experimental |
@msaroufim I think cpu related work is something we are already talking about moving out of experimental. @drisspg no specific objection of experimental vs prototype. Just curious on your motivation |
I was taking a look at the repo structure of torchao and I think the experimental naming and location is quite confusing. We just did some house keeping and I think that any effort to clean up the codebase is valuable |
Agreed, but IMO let's move out of experimental first (fast, one PR), and figure out how to graduate later (larger effort).
We should be consistent throughout the repo to minimize confusion, the prototype folder is where all the other similar projects live. @kimishpatel could you help find someone to actually do this soon? |
Yeah at the time of introducing experimental folder, prototype one had some different semantics. If we agree that both are the same. i am ok to do the move. I have some code in pipe though that i would like to land before I can get @metascroy or someone to help |
A couple things:
|
sgtm! |
@kimishpatel , wondering if you can share some context? I honestly never understood why the code was added to |
On more thing to add: I don't know how smooth the migration from experimental to prototype will be. It will require updating a lot of C++ include paths, and it might break internal tests because I don't think prototype is mirrored in xplat like experimental is. |
@vkuzo I dont remember all the context, but it was more of "experimental" was something no one else landed their stuff except us. And maybe that was the reason to avoid contamination. I dont think it was anything beyond that. |
Summary
The naming / project structure is confusing I think we should move this project to Prototype folder
The text was updated successfully, but these errors were encountered: