Free cookie consent management tool by TermsFeed Policy Generator

Opened 8 years ago

Closed 8 years ago

#2525 closed defect (done)

Some types are missing a StorableClassAttribute

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

Description (last modified by pfleck)

Some types do not have a StorableClassAttribute although they have storable members or a storable constructor, e.g. SimpleSymbolicExpressionGrammar.

If a type contains a storable member but has no StorableClassAttribute, it cannot be persisted, e.g. OptimizationSupportScript<T> of an ExternalEvaluationProblem.

Change History (8)

comment:1 Changed 8 years ago by pfleck

r13392 Added missing StorableClassAttribute.

comment:2 Changed 8 years ago by pfleck

r13393 Added a unit test that reports types that have a Storable member but does not define a StorableClassAttribute.

comment:3 Changed 8 years ago by pfleck

  • Description modified (diff)
  • Status changed from new to accepted
  • Summary changed from ExternalEvaluationProblem cannot be persisted to Some types are missing a StorableClassAttribute

comment:4 Changed 8 years ago by pfleck

  • Milestone changed from HeuristicLab 4.0 to HeuristicLab 3.3.14

r13395

  • Changed unit test that it also lists types that have a StorableConstructor but are not marked as StorableClass.
  • Put both storable tests in a single StorableTest.cs.
  • Added missing StorableClassAttributes.

comment:5 Changed 8 years ago by pfleck

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

comment:6 Changed 8 years ago by mkommend

Thanks for implementing this test.

Reviewed r13392, r13393, and r13395.

comment:7 Changed 8 years ago by mkommend

  • Status changed from reviewing to readytorelease

comment:8 Changed 8 years ago by mkommend

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

r13636: Merged r13392, r13393 and r13395 into stable.

Note: See TracTickets for help on using tickets.