Opened 4 weeks ago

Last modified 3 weeks ago

#3005 readytorelease feature request

There should be an option to make ValueParameters read only

Reported by: mkommend Owned by: mkommend
Priority: medium Milestone: HeuristicLab 3.3.16
Component: Parameters Version: trunk
Keywords: Cc:

Description

Especially in base classes the use of parameters is not clear upfront. However, in subclasses (e.g. concrete problem implementations) the value of a parameter should never be deleted / overriden. Therefore, a ready only flag which prevents modification of the parameter value per code and in the GUI should be introduced.

Change History (9)

comment:1 Changed 4 weeks ago by mkommend

  • Status changed from new to accepted

comment:2 Changed 4 weeks ago by mkommend

r16872: Added readonly flag to all value parameters.

comment:3 Changed 4 weeks ago by mkommend

r16873: Adapted genetic programming problems to disallow modifications of the grammar.
r16875: Removed parameter ctors with getsCollected flag.

comment:4 Changed 4 weeks ago by mkommend

  • Owner changed from mkommend to abeham
  • Status changed from accepted to reviewing

comment:5 Changed 4 weeks ago by abeham

  • Owner changed from abeham to mkommend
  • Status changed from reviewing to readytorelease

Reviewed r16872

Setting the readonly parameter to true after assigning a grammar in UpdateGrammar may override that a user of that class has previously set it to false.

comment:6 Changed 4 weeks ago by mkommend

r16877: Adapted FLA to the changed parameter ctors.

comment:7 Changed 3 weeks ago by bburlacu

r16890: Add null check to prevent exception in ConstrainedValueParameterView

comment:8 Changed 3 weeks ago by gkronber

ad r16877: visual studio auto-format settings problem

comment:9 Changed 3 weeks ago by abeham

r16916: reverted formatting changes of EngineAlgorithmOperator and added .editorconfig file to solution

Note: See TracTickets for help on using tickets.