[BUG] ReferenceError: weakly-referenced object no longer exists · voxel51/fiftyone#2738

(4 comments) (0 reactions) (0 assignees)Python (4,021 stars) (400 forks)batch import

bugcoregood first issue

説明

Here's a perplexing mongoengine-related error:

import fiftyone as fo
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset("quickstart")

# this fails
dataset.first()["tags"].append("tag")

# but this works
sample = dataset.first()
sample["tags"].append("tag")

ReferenceError                            Traceback (most recent call last)
<ipython-input-2-846545a97f03> in <module>
----> 1 dataset.first()['tags'].append('tag')

~/dev/env/fo/lib/python3.9/site-packages/mongoengine/base/datastructures.py in wrapper(self, *args, **kwargs)
     21         # Can't use super() in the decorator.
     22         result = parent_method(self, *args, **kwargs)
---> 23         self._mark_as_changed()
     24         return result
     25 

~/dev/env/fo/lib/python3.9/site-packages/mongoengine/base/datastructures.py in _mark_as_changed(self, key)
    179 
    180     def _mark_as_changed(self, key=None):
--> 181         if hasattr(self._instance, "_mark_as_changed"):
    182             if key is not None:
    183                 self._instance._mark_as_changed(f"{self._name}.{key % len(self)}")

ReferenceError: weakly-referenced object no longer exists

コントリビューターガイド

技術スタック: pythonmongodb
領域: backenddatabase
Issue 種別: bug
難度: 3
推定時間: 1-3 hours
活動状況: fresh
明確さ: clear
前提条件: Pythonmongoengine basics
初心者向け度: 55
調査方針: The error occurs when calling dataset.first()['tags'].append('tag') because the returned mongoengine list loses its weak reference to the parent document. Investigate the mark as changed method in mongoengine/base/datastructures.py, specifically how the instance reference is stored. The workaround (assigning to a variable) suggests a garbage collection issue. Look for similar patterns in the codebase and consider if the list should keep a strong reference.