mirror of
				https://github.com/explosion/spaCy.git
				synced 2025-11-04 09:57:26 +03:00 
			
		
		
		
	* fix: De/Serialize `SpanGroups` including the SpanGroup keys This prevents the loss of `SpanGroup`s that have the same .name as other `SpanGroup`s within the same `SpanGroups` object (upon de/serialization of the `SpanGroups`). Fixes #10685 * Maintain backwards compatibility for serialized `SpanGroups` (serialized as: a list of `SpanGroup`s, or b'') * Add tests for `SpanGroups` deserialization backwards-compatibility * Move a `SpanGroups` de/serialization test (test_issue10685) to tests/serialize/test_serialize_spangroups.py * Output a warning if deserializing a `SpanGroups` with duplicate .name-d `SpanGroup`s * Minor refactor * `SpanGroups.from_bytes` handles only `list` and `dict` types with `dict` as the expected default * For lists, keep first rather than last value encountered * Update error message * Rename and update tests * Update to preserve list serialization of SpanGroups To avoid breaking compatibility of serialized `Doc` and `DocBin` with earlier versions of spacy v3, revert back to a list-only serialization, but update the names just for serialization so that the SpanGroups keys override the SpanGroup names. * Preserve object identity and current key overwrite * Preserve SpanGroup object identity * Preserve last rather than first span group from SpanGroup list format without SpanGroups keys * Update inline comments * Fix types * Add type info for SpanGroup.copy * Deserialize `SpanGroup`s as copies when a single SpanGroup is the value for more than 1 `SpanGroups` key. This is because we serialize `SpanGroups` as dicts (to maintain backward- and forward-compatibility) and we can't assume `SpanGroup`s with the same bytes/serialization were the same (identical) object, pre-serialization. * Update spacy/tokens/_dict_proxies.py * Add more SpanGroups serialization tests Test that serialized SpanGroups maintain their Span order * small clarification on older spaCy version * Update spacy/tests/serialize/test_serialize_span_groups.py Co-authored-by: Adriane Boyd <adrianeboyd@gmail.com> Co-authored-by: Sofie Van Landeghem <svlandeg@users.noreply.github.com>
		
			
				
	
	
		
			110 lines
		
	
	
		
			4.4 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
			
		
		
	
	
			110 lines
		
	
	
		
			4.4 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
from typing import Dict, Iterable, List, Tuple, Union, Optional, TYPE_CHECKING
 | 
						|
import warnings
 | 
						|
import weakref
 | 
						|
from collections import UserDict
 | 
						|
import srsly
 | 
						|
 | 
						|
from .span_group import SpanGroup
 | 
						|
from ..errors import Errors, Warnings
 | 
						|
 | 
						|
 | 
						|
if TYPE_CHECKING:
 | 
						|
    # This lets us add type hints for mypy etc. without causing circular imports
 | 
						|
    from .doc import Doc  # noqa: F401
 | 
						|
    from .span import Span  # noqa: F401
 | 
						|
 | 
						|
 | 
						|
# Why inherit from UserDict instead of dict here?
 | 
						|
# Well, the 'dict' class doesn't necessarily delegate everything nicely,
 | 
						|
# for performance reasons. The UserDict is slower but better behaved.
 | 
						|
# See https://treyhunner.com/2019/04/why-you-shouldnt-inherit-from-list-and-dict-in-python/
 | 
						|
class SpanGroups(UserDict):
 | 
						|
    """A dict-like proxy held by the Doc, to control access to span groups."""
 | 
						|
 | 
						|
    _EMPTY_BYTES = srsly.msgpack_dumps([])
 | 
						|
 | 
						|
    def __init__(
 | 
						|
        self, doc: "Doc", items: Iterable[Tuple[str, SpanGroup]] = tuple()
 | 
						|
    ) -> None:
 | 
						|
        self.doc_ref = weakref.ref(doc)
 | 
						|
        UserDict.__init__(self, items)  # type: ignore[arg-type]
 | 
						|
 | 
						|
    def __setitem__(self, key: str, value: Union[SpanGroup, Iterable["Span"]]) -> None:
 | 
						|
        if not isinstance(value, SpanGroup):
 | 
						|
            value = self._make_span_group(key, value)
 | 
						|
        assert value.doc is self.doc_ref()
 | 
						|
        UserDict.__setitem__(self, key, value)
 | 
						|
 | 
						|
    def _make_span_group(self, name: str, spans: Iterable["Span"]) -> SpanGroup:
 | 
						|
        doc = self._ensure_doc()
 | 
						|
        return SpanGroup(doc, name=name, spans=spans)
 | 
						|
 | 
						|
    def copy(self, doc: Optional["Doc"] = None) -> "SpanGroups":
 | 
						|
        if doc is None:
 | 
						|
            doc = self._ensure_doc()
 | 
						|
        return SpanGroups(doc).from_bytes(self.to_bytes())
 | 
						|
 | 
						|
    def setdefault(self, key, default=None):
 | 
						|
        if not isinstance(default, SpanGroup):
 | 
						|
            if default is None:
 | 
						|
                spans = []
 | 
						|
            else:
 | 
						|
                spans = default
 | 
						|
            default = self._make_span_group(key, spans)
 | 
						|
        return super().setdefault(key, default=default)
 | 
						|
 | 
						|
    def to_bytes(self) -> bytes:
 | 
						|
        # We serialize this as a dict in order to track the key(s) a SpanGroup
 | 
						|
        # is a value of (in a backward- and forward-compatible way), since
 | 
						|
        # a SpanGroup can have a key that doesn't match its `.name` (See #10685)
 | 
						|
        if len(self) == 0:
 | 
						|
            return self._EMPTY_BYTES
 | 
						|
        msg: Dict[bytes, List[str]] = {}
 | 
						|
        for key, value in self.items():
 | 
						|
            msg.setdefault(value.to_bytes(), []).append(key)
 | 
						|
        return srsly.msgpack_dumps(msg)
 | 
						|
 | 
						|
    def from_bytes(self, bytes_data: bytes) -> "SpanGroups":
 | 
						|
        # backwards-compatibility: bytes_data may be one of:
 | 
						|
        # b'', a serialized empty list, a serialized list of SpanGroup bytes
 | 
						|
        # or a serialized dict of SpanGroup bytes -> keys
 | 
						|
        msg = (
 | 
						|
            []
 | 
						|
            if not bytes_data or bytes_data == self._EMPTY_BYTES
 | 
						|
            else srsly.msgpack_loads(bytes_data)
 | 
						|
        )
 | 
						|
        self.clear()
 | 
						|
        doc = self._ensure_doc()
 | 
						|
        if isinstance(msg, list):
 | 
						|
            # This is either the 1st version of `SpanGroups` serialization
 | 
						|
            # or there were no SpanGroups serialized
 | 
						|
            for value_bytes in msg:
 | 
						|
                group = SpanGroup(doc).from_bytes(value_bytes)
 | 
						|
                if group.name in self:
 | 
						|
                    # Display a warning if `msg` contains `SpanGroup`s
 | 
						|
                    # that have the same .name (attribute).
 | 
						|
                    # Because, for `SpanGroups` serialized as lists,
 | 
						|
                    # only 1 SpanGroup per .name is loaded. (See #10685)
 | 
						|
                    warnings.warn(
 | 
						|
                        Warnings.W120.format(
 | 
						|
                            group_name=group.name, group_values=self[group.name]
 | 
						|
                        )
 | 
						|
                    )
 | 
						|
                self[group.name] = group
 | 
						|
        else:
 | 
						|
            for value_bytes, keys in msg.items():
 | 
						|
                group = SpanGroup(doc).from_bytes(value_bytes)
 | 
						|
                # Deserialize `SpanGroup`s as copies because it's possible for two
 | 
						|
                # different `SpanGroup`s (pre-serialization) to have the same bytes
 | 
						|
                # (since they can have the same `.name`).
 | 
						|
                self[keys[0]] = group
 | 
						|
                for key in keys[1:]:
 | 
						|
                    self[key] = group.copy()
 | 
						|
        return self
 | 
						|
 | 
						|
    def _ensure_doc(self) -> "Doc":
 | 
						|
        doc = self.doc_ref()
 | 
						|
        if doc is None:
 | 
						|
            raise ValueError(Errors.E866)
 | 
						|
        return doc
 |