Skip to content

swarm.points physical-coordinate scaling mislabels units (meter magnitudes labelled with mesh.units) #386

Description

@lmoresi

Found during the #379/#382 adversarial review (PR #382 discussion). The deprecated swarm.points / swarm.data snapshot scales model coordinates by CoordinateSystem._length_scale and wraps the result in UnitAwareArray(..., units=self.mesh.units). When the mesh units are kilometres, the magnitudes produced by the scale factor are metres — a 1000x label/magnitude mismatch. Verified in the review's regime battery (max|diff| ~2.7e6 against the correct mesh-variable path). Pre-existing behaviour, faithfully preserved by the #382 read-only snapshot; the sanctioned swarm.coords path dimensionalises correctly and is unaffected.

Underworld development team with AI support from Claude Code

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions