Skip to content
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

Reorganize and unify naming for all reactive operators #1043

Closed
glopesdev opened this issue Sep 19, 2022 · 0 comments · Fixed by #1046
Closed

Reorganize and unify naming for all reactive operators #1043

glopesdev opened this issue Sep 19, 2022 · 0 comments · Fixed by #1046
Labels
proposal Request for a new feature
Milestone

Comments

@glopesdev
Copy link
Member

The current namespace organization is messy for nested reactive operators, as they are split between Bonsai.Expressions and Bonsai.Reactive. It would be more clear to have everything that is not purely expression manipulation inside Bonsai.Reactive. We can also drop the Builder suffix for all nodes which are moved.

Finally, for consistency we might want to rename operator variants with a suffix, rather than prefix (e.g. WindowCount instead of ElementCountWindow).

@glopesdev glopesdev added the proposal Request for a new feature label Sep 19, 2022
@glopesdev glopesdev added this to the 2.7.0 milestone Sep 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal Request for a new feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant