the Sliver.slices parameter needs improvement

[chaos@bbn.com]

The Sliver class has a parameter called slices, which is a list. There are a few problems with this:

1. There is no setter for this parameter
2. A sliver can belong to at most one slice, so this should be a single parameter, not a list
3. I don't think it should be legal to define a sliver with no slice, so this should be a mandatory parameter

[chaos@bbn.com]

Fully completing 3 may require the ability to query the GMOC server for slices, since, in general, an aggregate will create Sliver objects, and will not be the one creating Slice objects from scratch.

[chaos@bbn.com]

This bug is still active. There is now a slices setter in Sliver. (I haven't tested it.)

However, 2 is still definitely active: a sliver belongs to one slice, so this should not be a list unless i am misunderstanding something about what this parameter is for. Am i?

[chaos@bbn.com]

In v0.2.1, there is only one slice per sliver (so 2 is fixed), but there is once again no setter for the property:

  slivers[sliver].slice = sinfo['slice_urn']

yields:

Traceback (most recent call last):
  File "./report_foam_relations", line 243, in <module>
    slivers[sliver].slice = sinfo['slice_urn']
AttributeError: can't set attribute

[chaos@bbn.com]

In v0.3.0, the slice setter is defined, but is a noop. The code:

  print "before: %s" % slivers[sliver].slice
  slivers[sliver].slice = slices[sinfo['slice_urn']]
  print "after:  %s" % slivers[sliver].slice

yields:

before: None
after:  None

So this bug is still active.

[chaos@bbn.com]

Item 2 is fixed: now:

  print "before: %s" % slivers[sliver].slice
  slivers[sliver].slice = slices[sinfo['slice_urn']]
  print "after:  %s" % slivers[sliver].slice

yields:

before: None
after:  <gmoc.Slice object at 0xa2e47ac>

Good enough.