Skip to content

Initial comments #2

Description

@kvid

I like this sandbox to play with new ideas.

These are my initial comments during my first reading of the committed code:

misc.py#L2: Optional is not needed unless we need to allow None as a legal value.

misc.py#L16: Is ColorScheme a possible alternative name for this class? I believe I have seen the color code term also being used for the RGB or HSV values.

links.py: I suggest a new class to be used for each pair in the Connection and MatePin classes:

class EndPoint:
    id: Designator
    port: int

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions