taborlab / Iris

The Light Program Interface
BSD 3-Clause "New" or "Revised" License
12 stars 2 forks source link

Can't enter 4009 as intensity for constant function for some reason #124

Closed castillohair closed 9 years ago

castillohair commented 9 years ago

Open website Add constant function Write 4009 in "Comma separated list of intensities" The website rejects the entry. The same happens for any 40xy, where 0<=x<=9 and 6<=y<=9. It seems that at some point you're comparing strings instead of numbers to check whether the number is < 4095.

SynbioLucas commented 9 years ago

Twas a problem with the regex validation. This should fix it once we have a stable version of the HTML:

[\[|\{|\(]?((?:40(?:5[0-9]|[0-9][0-5])|40[0-8][0-9]|1[0-9]{3}|2[0-9]{3}|3[0-9]{3}|[0-9][0-9]{0,2}) *, *)*(?:40(?:5[0-9]|[0-9][0-5])|40[0-8][0-9]|1[0-9]{3}|2[0-9]{3}|3[0-9]{3}|[0-9][0-9]{0,2})[\]|\}|\)]?

SynbioLucas commented 9 years ago

Fixed in b0cb6db2d5ff7de24705daef65cd72487f491379 (newGUI)