-
Notifications
You must be signed in to change notification settings - Fork 31
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
Cutting a release #66
Comments
Yeah !!!!!!!!!!
…On Sat, Jan 8, 2022 at 4:55 PM Brian Chen ***@***.***> wrote:
Even though Diffractor isn't yet feature complete, it can already be a major
improvement
<FluxML/Flux.jl#1809 (comment)> over
Zygote for real-world use cases.
—
Reply to this email directly, view it on GitHub
<#66>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAI5LYDXALSQZY6V5AKRGRDUVCXEVANCNFSM5LRCB73A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Just to be clear, this is a question to the maintainers and not an announcement or guarantee of anything :) |
Right :-)
…On Sat, Jan 8, 2022 at 5:52 PM Brian Chen ***@***.***> wrote:
Just to be clear, this is a question to the maintainers and not an
announcement or guarantee of anything :)
—
Reply to this email directly, view it on GitHub
<#66 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAI5LYDFMXCAOO2R55BQNRDUVC54TANCNFSM5LRCB73A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you commented.Message ID:
***@***.***>
|
(lesson for me: not to reply via e-mail interface :-) ) |
we should cut a 0.1.0 release. Probably should do so as soon as CI is not failing? |
Not blocking, but should we make some effort to start organising a larger set of tests for this package? Zygote's tests contain a lot of knowledge about potentially awkward edge cases, and it's possible that we should trawl through them and capture that. While hopefully leaving behind some of the very repetitive bits, or things which seem well captured by ChainRules's tests. It has few second derivative tests, and maybe no forward mode tests, but the same awkward cases should ideally work for all of these. Perhaps we should structure things to make that easy, even if many fail right now. |
I think copying a lot of Zygote's tests would be a good idea, and just mark them broken if they don't work. That's something anyone interested in Diffractor can help with. |
Even though Diffractor isn't yet feature complete, it can already be a major improvement over Zygote for real-world use cases.
The text was updated successfully, but these errors were encountered: