-
Notifications
You must be signed in to change notification settings - Fork 149
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
The state of 0.5 version? #75
Comments
@qrilka The basic plan is to get some of the branches that have been kicking around merged in, then some testing and release. The branches have fairly big performance improvements, so I don't want to pass them by. Also, I need to do a bit more to ensure that 0.5 is as compatible with 0.4 as possible - I don't want people to have to change their code too much to upgrade. |
Thanks for your quick response @ryantrinkle |
Joining the queue for the Christmas present :) On Fri, 4 Nov 2016, 13:59 Kirill Zaborsky, notifications@github.com wrote:
Lingnan Dai |
@ryantrinkle is it expected for the current |
@qrilka Are you referring to No, I wouldn't expect increased memory usage like that - it would be great to have an issue with the info to reproduce that. Would be very interesting to track down! |
oh, sorry... |
Filed #81 regarding memory issues |
Also I've noticed that porting to 0.5.0 increased the size of our minified app js by 200k - is it expected? |
@ryantrinkle, maybe making a GitHub milestone for 0.5.0 release could help us? |
Any update on this? It'd be nice to get an official release. |
@sboosali Agreed. @deepfire A github milestone is a great idea; I will try to prioritize creating one with everything that's needed for a release ASAP. Hopefully it will be fairly small things that are required. Off the top of my head:
|
while pretty trivial, w.r.t. missing instances, FunctorMaybe is satsified by several containers, anything that can be empty. Like:
also, the current Witherable seems to have a Filterable class with only a Functor super class. https://hackage.haskell.org/package/witherable-0.2/docs/Data-Witherable.html If it doesn't add any dependencies not already present, maybe it's worth it to just pull it in? |
@sboosali Using something outside reflex for FunctorMaybe would be great; however, there are a few options, and I'm not quite sure which is best. What about Compactable? Do you have any sense of how it compares with witherable? |
@deepfire @sboosali Added the new milestone here: https://github.com/reflex-frp/reflex/milestone/1 |
I like that Witherable subclasses Functor; Compactable doesn't. It's more
lawful, and will keep the constraints shorter. (But both provide default
method definitions on Functors.) Also, Witherable has more downloads, and fewer dependencies.
But, I haven't used the `fmapMaybe` concept beyond mapMaybe, so I can't
say, they both look fine.
…On Jan 3, 2018 5:23 AM, "Ryan Trinkle" ***@***.***> wrote:
@deepfire <https://github.com/deepfire> @sboosali
<https://github.com/sboosali> Added the new milestone here:
https://github.com/reflex-frp/reflex/milestone/1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#75 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACNoMXC-xVerDi6Q6L61p0z-Wfza_LHqks5tG39ugaJpZM4KpiqP>
.
|
@sboosali Compactable subsumes Witherable. So I recommend Compactable (not just because I wrote it), but because you can't use Witherable with |
Event is not Witherable, just Filterable.
…On Jan 3, 2018 10:30 AM, "Isaac Shapira" ***@***.***> wrote:
@sboosali <https://github.com/sboosali> Compactable subsumes Witherable.
So I recommend Compactable (not just because I wrote it), but because you
can't use Witherable with Event t, since it has extraneous constraints.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#75 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACNoMRGKXr_jWOoZW7p_Ex0etWo5Mfg8ks5tG8dSgaJpZM4KpiqP>
.
|
@sboosali right. But if you use Compactable, |
I agree with @sboosali. The
|
yeah, I agree about ` f (Maybe a) -> f a ` :-)
On Jan 3, 2018 10:34 AM, "Isaac Shapira" <notifications@github.com> wrote:
@sboosali <https://github.com/sboosali> right. But if you use
Compactable, Event
t is Compactable, as well as can have a Compact Functor instance, and
filter comes for free. Imho Filterable is a broken abstraction, compact ::
f (Maybe a) -> f a is the core abstraction. All the Witherable abstractions
can be written in terms of compact, but not vice versa.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#75 (comment)>,
or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ACNoMaDXlBTuuhhi94ib5U-_Tp1Njkoiks5tG8hEgaJpZM4KpiqP>
.
|
@ElvishJerricco Very good point about specialization. Since ghcjs is still on 8.0, and 8.0 doesn't seem to ever specialize code when a method has a constraint, I think we should defer this discussion until ghcjs moves to 8.2 and we re-test. Unspecialized code is insanely slow. |
I got rid of one of the missing method implementations today, leaving just traverseIntMapWithKeyWithAdust for QueryT. I'm also most of the way through assigning issues to milestones. Progress! |
Alright, every issue now has a milestone: either v0.5 or Later. |
You could just push out a release and set the milestones for v0.6. Looking at the haddock difference 0.4 is hopelessly out of date. |
Hi everyone - closing this as 0.5 has hit hackage. Sorry for the wait. |
Congratulations, gentlemen, and thank you for your great work! : -) |
I think it (version 0.5) is in the
master
for some time already but I have seen no plans or roadmap - are there any?The text was updated successfully, but these errors were encountered: