Should there be a way to disable trait notifications?

Offen

#907 erstellt am 16.7.2024 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 5/5
Geschätzter Aufwand: Über eine Woche
Anfängerfreundlichkeit: 25/100

Issue-Typ: Feature
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: backend

Rechercherichtung

Beginnen Sie mit dem Lesen der vorhandenen HasTraits-Benachrichtigungsimplementierung und des im Issue referenzierten Kontextmanagers hold_trait_notifications. Ermitteln Sie, ob ein Kontextmanager zum vollständigen Deaktivieren in die öffentliche API gehört, und definieren Sie sein erwartetes Wiederherstellungsverhalten; abgeschlossen wäre dies mit einer vereinbarten API und den entsprechenden Tests oder der entsprechenden Dokumentation.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

I recently had to completely disable trait notification during instantiation of a HasTraits child class. Holding with hold_trait_notifications wasn't sufficient; the notifications release when exiting the context manager was modifying attributes I didn't want to change.

I used a snippet found in the source code to implement the following context manager.

import contextlib

from traitlets import HasTraits, Bunch


@contextlib.contextmanager
def disable_trait_notifications(self):
    """Temporatilly disable trait notifications."""
    def ignore(change: Bunch) -> None:
        pass
    original_notify_change = self.notify_change
    self.notify_change = ignore
    try:
        yield
    finally:
        self.notify_change = original_notify_change

# Monkey patch HasTraits
HasTraits.disable_trait_notifications = disable_trait_notifications

Wondering if it would make sense to have this as part of the API. Hope I'm not suggesting an anti pattern 😅

Vorherrschende Sprache: Python
Sterne: 653
Forks: 217
Ø Merge: 2 T. 21 Std.
Gemergte PRs (30 T.): 2

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus ipython/traitlets

docs: document `log` trait on `Application` class Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 65/100

ipython/traitlets#755 · 1 Reaktion · Erstellt am 06.7.2022
Better insight on dictionary of traits instantiation Offen

Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 65/100

ipython/traitlets#549 · Erstellt am 26.4.2020
Add Deprecation of Trait Declarations via Classes to Migration Notes Offen

good first issue

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 85/100

ipython/traitlets#520 · Erstellt am 04.4.2019
Document `TraitType.read_only` Offen

documentation

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 65/100

ipython/traitlets#410 · Erstellt am 31.5.2017
traitlets.Set should support generic type annotations for type safety Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100

ipython/traitlets#927 · 1 Reaktion · Erstellt am 27.6.2025

Alle Issues in ipython/traitlets