Skip to content

Change Monitor dict to a proper class #470

@halldorfannar

Description

@halldorfannar

I suggest we do this in the next major release. Since Monitor now contains heterogenous types (int, bool, str) it feels quite messy to have it a dict. When converting it to a class we could also make the attributes read-only, simply by having it inherit from a NamedTuple. I think this would be quite neat. But it's just a suggestion.

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions