Opened 4 years ago

Closed 4 years ago

#2045 closed enhancement (done)

Minor DataAnalysisImportDialog Improvements

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

Description

The import dialog should not catch all exceptions, also some anchors are not set correctly.

The ClassificationProblemData can only use data with at most 99 classes instead of the specified 100.

ProblemInstanceConsumerViewGeneric should not be a default view, because it might be set in the view host instead of the ProblemView, if there is no more specific view.

Change History (11)

comment:1 Changed 4 years ago by sforsten

  • Status changed from new to accepted

comment:2 Changed 4 years ago by sforsten

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

r9449: fixed described problems

comment:3 Changed 4 years ago by mkommend

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

comment:4 Changed 4 years ago by gkronber

  • Owner changed from sforsten to gkronber
  • Status changed from readytorelease to assigned

comment:5 Changed 4 years ago by gkronber

r9457: partial revert of r9449 in TableFileParser (caused a unit test failure).

comment:6 Changed 4 years ago by gkronber

  • Owner changed from gkronber to sforsten
  • Status changed from assigned to reviewing

comment:7 Changed 4 years ago by sforsten

Either the DataAnalysisImportDialog has also to be adapted not to use spaces as separators or the unit test has to be changed.

My suggestion is to change the unit test and make it possible to also use spaces as separators in files.

comment:8 Changed 4 years ago by sforsten

r9458: removed the option to select spaces as separators in DataAnalysisImportTypeDialog

comment:9 Changed 4 years ago by sforsten

  • Owner changed from sforsten to mkommend

comment:10 Changed 4 years ago by mkommend

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

Reviewed r9457 & r9458.

comment:11 Changed 4 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.