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

Remove recompute action #43

Closed
cowboyd opened this issue Oct 24, 2016 · 4 comments
Closed

Remove recompute action #43

cowboyd opened this issue Oct 24, 2016 · 4 comments

Comments

@cowboyd
Copy link
Member

cowboyd commented Oct 24, 2016

Now that we have set, and we understand that the general form of a microstate is (Microstate value options) we don't have the need for this to be an action. If custom initialization needs to be handled, it should be done in a method like construct, initialize, recomputeState, etc...

@taras
Copy link
Member

taras commented Oct 24, 2016

What about a reset transition that transitions the microstate into value that's initialized from passed in arguments?

@cowboyd
Copy link
Member Author

cowboyd commented Oct 24, 2016

I think that can wait until a subsequent release. We don't have any functionality that depends on this.

@taras
Copy link
Member

taras commented Oct 24, 2016

Do you want to remove recompute before 0.3.0?

@cowboyd
Copy link
Member Author

cowboyd commented Oct 25, 2016

I think we should since it's currently a public api that is callable from the templates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants