search

cvhciKIT / sloth

Sloth is a tool for labeling image and video data for computer vision research.
Other
610 stars 198 forks source link

deleting keys from the dictionary actually cripples the dict in the container #3

Closed baeuml closed 13 years ago

mika-fischer commented 13 years ago

I thought the point was that the container should not have a dictionary, no?

baeuml commented 13 years ago

agreed. I was thinking about whether the container might need to now something about the annotations in order to correctly resolve image path names, but that is probably not necessary. Otherwise we would need to propagate model changes to the container, too, which I believe we agree is not something we want...

However, then we should remove the annotation state from the default container, and just let load() return the annotations, and let save() have another parameter for a passing the annotations.

Plus, we should probably rename the container to something more appropriate that does not suggest that it actually contains anything...

baeuml commented 13 years ago

see #4 and #5