Opened 11 years ago
Closed 11 years ago
#2110 closed defect (done)
Import/Export for ProblemInstanceConsumers of multiple data types is not shown
| Reported by: | svonolfe | Owned by: | abeham |
|---|---|---|---|
| Priority: | medium | Milestone: | HeuristicLab 3.3.9 |
| Component: | Problems.Instances | Version: | 3.3.8 |
| Keywords: | Cc: | abeham |
Description (last modified by abeham)
The ProblemInstanceConsumerView<T> is instantiated with one of the concrete types that a problem supports, thus the GenericConsumer, GenericExporter, and GenericSelectedProvicer are null.
E.g. the QuadraticAssignmentProblem supports QAPData as well as TSPData. It thus implements both IProblemInstanceConsumer<QAPData> and IProblemInstanceConsumer<TSPData>. The view that will list libraries for both data types is instantiated erroneously as an ProblemInstanceConsumerView<QAPData>.
The best solution to this problem has been identified to move all provider related functionality (import, export, help display) to the provider view which is always specific.
Change History (13)
comment:1 Changed 11 years ago by svonolfe
- Status changed from new to accepted
comment:2 Changed 11 years ago by abeham
- Component changed from Problems.VehicleRouting to Problems.Instances
- Description modified (diff)
- Milestone changed from HeuristicLab 3.3.x Backlog to HeuristicLab 3.3.9
- Owner changed from svonolfe to abeham
- Status changed from accepted to assigned
- Summary changed from Import button is not visible for some VRP variants to Import/Export for ProblemInstanceConsumers of multiple data types is not shown
comment:3 Changed 11 years ago by abeham
comment:4 Changed 11 years ago by abeham
- Owner changed from abeham to jkarder
comment:5 Changed 11 years ago by jkarder
- Status changed from assigned to accepted
comment:6 Changed 11 years ago by jkarder
r10000: refactored ProblemInstanceProviderViews
comment:7 Changed 11 years ago by jkarder
comment:8 Changed 11 years ago by jkarder
r10003: fixed ProblemView to hide all provider related controls if its content is no IProblemInstanceConsumer
comment:9 Changed 11 years ago by jkarder
- Owner changed from jkarder to abeham
- Status changed from accepted to reviewing
comment:10 Changed 11 years ago by jkarder
r10005: adapted view in MetaOpt branch
comment:11 Changed 11 years ago by abeham
r10009: renamed generic view class
comment:12 Changed 11 years ago by abeham
- Status changed from reviewing to readytorelease
comment:13 Changed 11 years ago by abeham
- Resolution set to done
- Status changed from readytorelease to closed
This ticket should be released to stable only after #1508 due to changes in r9991 which likely also affect this ticket.