Opened 11 months ago

Closed 9 months ago

#2593 closed enhancement (done)

Add method to convert a Dataset into a Modifiable dataset and possibility to replace columns

Reported by: bburlacu Owned by: mkommend
Priority: medium Milestone: HeuristicLab 3.3.14
Component: Problems.DataAnalysis Version: 3.3.13
Keywords: Cc:

Description

The default ModifiableDataset copy constructor does not copy the values, leading to potential problems when two datasets reference the same values. A factory method for the conversion Dataset -> ModifiableDataset (with value cloning) would be a better option.

Additionally, it would be more efficient to be able to replace entire columns instead of individually changing variable values per row.

Change History (5)

comment:1 Changed 11 months ago by bburlacu

  • Status changed from new to accepted

comment:2 Changed 11 months ago by bburlacu

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

r13760: Add ToModifiable method to the Dataset class. Add ReplaceColumn method to the ModifiableDataset.

comment:3 Changed 11 months ago by bburlacu

r13761: Rename ReplaceColumn method to ReplaceVariable and make it non-generic. Use new list constructor instead of extension method in the ToModifiable method.

comment:4 Changed 9 months ago by mkommend

  • Status changed from reviewing to readytorelease

Reviewed r13760 and r13761.

comment:5 Changed 9 months ago by mkommend

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

r13949: Merged r13760 and r13761 into stable.

Note: See TracTickets for help on using tickets.