search

UnePatate5010 / GUI-White-Black-Box-Models-Mix

Enhancing Machine Learning Interpretability Through a GUI
MIT License
1 stars 1 forks source link

Spinbox entries #4

Closed UnePatate5010 closed 3 months ago

UnePatate5010 commented 4 months ago

Currently some entries are spinboxes (+ and - button and text entry). Spinbox has three main problems:

  1. Everything can be written in the text entry (int, float, string...) but only int are supported (and in some cases 'None'). A wrong input results in a error.
  2. Float should be supported, in at least some cases.
  3. There is no maximum value. For some parameters it should (such as percentages).
UnePatate5010 commented 3 months ago

Points 2 and 3 (floats adn maximum value) were fixed in 5bc9db4. The first point is quite difficult to fix because it would need to dynamically check every time the spinbox is modified and detect if the new value is not composed of number and change its content back to the previous value or a default value, which seems really boring. Therefore I will not bother with it and consider that if the user inputs letters instead of numbers then it is their mistake.