Skip to content

Allow specifying the coincident-node merge tolerance #1770

Description

@rajeeja

Follow-up to #1692, which merges coincident nodes at Grid construction using ERROR_TOLERANCE (1e-8, a chord length on the unit sphere).

That value is not reachable from any public API. Raising it means calling _merge_coincident_grid_ds_nodes on grid_ds before Grid.__init__, and lowering it is not possible at all, since the constructor re-runs the merge at the default. Exposing it means threading a parameter through open_grid and the from_* constructors, which is why it was left out of #1692.

Originally requested in #865 (comment) and raised again by @Sevans711 in review.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions