Free cookie consent management tool by TermsFeed Policy Generator

Opened 11 years ago

Closed 11 years ago

#1981 closed defect (done)

The wrong ProblemInstanceProviderView is chosen for RegressionProblemInstances

Reported by: mkommend Owned by: abeham
Priority: medium Milestone: HeuristicLab 3.3.8
Component: Problems.Instances Version: 3.3.8
Keywords: Cc:

Description

Therefore the import and export buttons are not visible for RegressionProblems. The reason for this is that the generic parameter in the problem instance consumer is defined as contravariant, which is only needed in the VRP implementation. It was discussed with svonolfen that this behavior is changed to correct the view discovery.

Change History (7)

comment:1 Changed 11 years ago by mkommend

  • Status changed from new to accepted

comment:2 Changed 11 years ago by mkommend

r8904: Removed the contravariance of the generic parameter in the IProblemInstanceConsumer.

comment:3 Changed 11 years ago by mkommend

  • Owner changed from mkommend to svonolfe
  • Status changed from accepted to assigned

Please adapt the VRP implementation to the changes in r8904.

comment:4 Changed 11 years ago by svonolfe

  • Owner changed from svonolfe to mkommend
  • Status changed from assigned to reviewing

Adapted VRP implementation to changes of r8904 - instance consuming in r8905, Adapted VRP test to changes in r8906

Last edited 11 years ago by svonolfe (previous) (diff)

comment:5 Changed 11 years ago by mkommend

  • Owner changed from mkommend to abeham

Tested and reviewed changes in r8905. However, I am not sure if the changes in r8904 affect other problem instances.

comment:6 Changed 11 years ago by abeham

  • Status changed from reviewing to readytorelease

As far as I know we don't have further abstractions in the problem instance data classes.

comment:7 Changed 11 years ago by swagner

  • Resolution set to done
  • Status changed from readytorelease to closed
  • Version changed from 3.3.7 to 3.3.8
Note: See TracTickets for help on using tickets.