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

(4 评论) (0 反应) (0 负责人)Python (4,021 star) (400 fork)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
议题类型: 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.