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

Error should be raised when we try to set slider of output variable #19

Open
danieljaeim opened this issue Jun 14, 2019 · 1 comment
Open

Comments

@danieljaeim
Copy link
Contributor

Currently the application has functionality to set sliders during burst capture, but we are currently unwittingly able to set the slider for our output variable.

Take the function y = mx + b (great function, very chique)

Screen Shot 2019-06-14 at 10 56 44 AM

Once we try burst capturing, if we accidentally set the slider index to be 1, which refers to the expression itself, we end up erasing the expression by simply setting the value of the expression to be the slider value.

Screen Shot 2019-06-14 at 10 59 34 AM

It would be nice if we could raise an error message after detecting whether our slider index is set to an expression, rather than a slider. This might require further information about how the calculator parses expressions, but otherwise we should be able to set an error if we try manipulating a slider value onto an expression!

@japamat0
Copy link
Contributor

japamat0 commented Jun 17, 2019

@alissarenz @danieljaeim Does it make sense to to provide the user with a list of valid variables which can be adjusted for gif output instead of validating the index?

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