-
Notifications
You must be signed in to change notification settings - Fork 3
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
[N/A] Clarify prebaked
docstring
#42
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: mhidalgo-bdai <[email protected]>
I later noticed that there is basically the same description here:
|
Very similar but not exactly the same. |
@mhidalgo-bdai should the descriptions of |
@ggoretkin-bdai descriptions are different already, but by omission. If it helps, we can definitely mention each other and make that difference explicit. |
@mhidalgo-bdai We should make the difference explicit. Otherwise, I doubt this will be the last time there is confusion |
@ggoretkin-bdai @mhidalgo-bdai Is this PR still useful or should we close it? |
@amessing-bdai @ggoretkin-bdai I think this still applies, provided the suggested docstring is in fact clearer to folks. |
Could someone please rebase and merge on my behalf if it is indeed useful? Thanks. |
No description provided.