Opened 4 years ago

Closed 3 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 4 years ago by svonolfe

  • Status changed from new to accepted

comment:2 Changed 4 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 4 years ago by abeham

This ticket should be released to stable only after #1508 due to changes in r9991 which likely also affect this ticket.

comment:4 Changed 4 years ago by abeham

  • Owner changed from abeham to jkarder

comment:5 Changed 4 years ago by jkarder

  • Status changed from assigned to accepted

comment:6 Changed 4 years ago by jkarder

r10000: refactored ProblemInstanceProviderViews

comment:7 Changed 4 years ago by jkarder

r10001: additional commit for r10000 (fix build fail)

comment:8 Changed 4 years ago by jkarder

r10003: fixed ProblemView to hide all provider related controls if its content is no IProblemInstanceConsumer

comment:9 Changed 4 years ago by jkarder

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

comment:10 Changed 3 years ago by jkarder

r10005: adapted view in MetaOpt branch

comment:11 Changed 3 years ago by abeham

r10009: renamed generic view class

comment:12 Changed 3 years ago by abeham

  • Status changed from reviewing to readytorelease

comment:13 Changed 3 years ago by abeham

  • Resolution set to done
  • Status changed from readytorelease to closed

r10021: merged r10000, r10001, r10003, r10005, r10009 to stable

Note: See TracTickets for help on using tickets.