Free cookie consent management tool by TermsFeed Policy Generator

Changes between Initial Version and Version 1 of Ticket #842, comment 28


Ignore:
Timestamp:
06/04/10 15:47:54 (14 years ago)
Author:
abeham
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #842, comment 28

    initial v1  
    11I wanted to add two more issues encountered when I was using the `ProgrammableOperator` recently:
    22 1. The parameters cannot be set.
    3  2. Using the default method parameter context in the code results in missing references.
     3 2. Using the parameter "context" easily results in missing references.
    44
    55Regarding 1.: The Execute method signature contains the values of the parameters, but not the parameters themselves. This means that if I design a custom evaluator with a Quality parameter, the value will be null (Quality is not yet available) and it cannot be set through (typically) !QualityParameter.!ActualValue = ... as only the value is given in the signature. It would be better to have the parameter itself in the method signature instead of its !ActualValue.
     
    77Regarding 2.: E.g. the simple line "int count = context.Scope.!SubScopes.Count;" requires that I add !HeuristicLab.Collections.!ObservableList as well as System.!ComponentModel.INotifyPropertyChanged. I would suggest including a bigger set of assemblies that are loaded by default.
    88
    9 There are also a view usability issues (from my point of view):
     9There are also a few usability issues (from my point of view):
    1010 1. There is no indication of what the two tree views on the left contain. A label "Assemblies" and "Namespaces" should probably be included.
    1111 2. The first tree view lists all !HeuristicLab plugins/assemblies first and groups everything else under "other", while the second tree view lists all other namespaces first and then groups the !HeuristicLab ones at the end. It would be better if both lists had the same grouping and order.