amuse icon indicating copy to clipboard operation
amuse copied to clipboard

ParticlesSuperset savepoint

Open ipelupessy opened this issue 6 years ago • 2 comments

Is your feature request related to a problem? Please describe. this is indeed related to #392 (thank you github). in short ParticleSupersets do not have savepoint. I am not sure it makes sense to have that for supersets, but for consistency it would be nice to have it (and otherwise a nice error message)

ipelupessy avatar Mar 20 '19 09:03 ipelupessy

there are at least two ways two implement this:

  • superset.savepoint generates savepoints on subsets, and then savepoints itself
  • superset copies subsets, referes to these and then savepoints itself (ie the original subsets are untouched)

ipelupessy avatar Mar 21 '19 13:03 ipelupessy

I think I'd have a preference for the second method, leaving the original subsets untouched.

rieder avatar Mar 21 '19 14:03 rieder