-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Selector Observable #84
Comments
Sorry, this slipped off my radar. I'll try to have a look at it later this week. |
That sounds reasonable, to me, but, IMO, it needs a better name. I think, Maybe |
Agreed that selector isn't a good name, I just grabbed that from behavior tree nomenclature. Heres yours and my suggestions combined:
let me know what you think of that last one, personally I like the idea of making it rather low level sounding because at least to me, |
I'm fine with |
Hi,
I have a combiner I've been shopping around for the right place to contribute to, I didn't notice anything in your library which appeared to do the same, but this seems like the appropriate place for such a thing:
https://gist.github.com/spaceribs/738ec30dac2874d5ba9f7718208410e9
Please take a look and let me know if it makes sense to put up a pull request for this feature.
The text was updated successfully, but these errors were encountered: