Skip to content

make mapper (actually) optional #393

@majidaldo

Description

@majidaldo

https://github.com/NREL/BuildingMOTIF/blob/64ae7a0cb49fdbf9f95fa07ab3592f43800bc40e/buildingmotif/ingresses/template.py#L26

I think the intent here was to not have the user need to provide a mapper. The sig should be mapper: Optional[Callable[[str], str]] = None, or, more simply mapper: Callable[[str], str]] = lambda s:s.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions