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

Factor min/max values are rounded to zero and break client #110

Closed
dk-teknologisk-mon opened this issue Sep 24, 2021 · 0 comments · Fixed by #111
Closed

Factor min/max values are rounded to zero and break client #110

dk-teknologisk-mon opened this issue Sep 24, 2021 · 0 comments · Fixed by #111
Labels
bug Something isn't working
Milestone

Comments

@dk-teknologisk-mon
Copy link

I've just tried entering a series of min/max values in the newui client, starting with two integer limits for minimal and maximal value (1 and 5), followed by slightly smaller values (0.5 and 1.5). This last case breaks the client (I'm guessing due to rounding) as the values that appear in the input model get changed to 0 and 1. If I go even lower to something like 0.03 and 0.15, both numbers are rounded to zero, which totally breaks the UI, with the name and values not making it up into the table of inputs (see image below).

image

If I set up the same experiment on the bee.boost.projects.alexandra UI, there is no issue with using "small" numbers for your factor limits, and you can work around the issue by defining the experiment there, saving it to .json and then import it in the newui client.

@dk-teknologisk-mon dk-teknologisk-mon added the bug Something isn't working label Sep 24, 2021
@langdal langdal linked a pull request Sep 24, 2021 that will close this issue
langdal added a commit that referenced this issue Sep 24, 2021
@langdal langdal added this to the v1.4.0 milestone Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants