Skip to content

[0.3.6] Develop to Main - #54

Merged
sujata-m merged 4 commits into
mainfrom
develop
Jun 10, 2026
Merged

[0.3.6] Develop to Main#54
sujata-m merged 4 commits into
mainfrom
develop

Conversation

@sujata-m

Copy link
Copy Markdown
Contributor

Dev Board Ticket

Changes

  • Fixed OSW→OSM ID remapping so ways preserve references to the exact ogr2osm node objects instead of resolving by duplicate temporary numeric IDs.
  • Prevented generated point-node IDs from being overwritten by OSW _id values, avoiding temporary node ID collisions before output.
  • Added BUG-3726 regression fixtures with real San Diego coordinates.
  • Added regression coverage to verify converted OSM ways keep local geometry and do not create cross-map spider lines.
  • Bumped package version to 0.3.6 and updated the changelog.

Testing

  • python -m unittest tests.unit_tests.test_osw2osm.test_osw2osm
  • python -m unittest discover tests/unit_tests

sujata-m and others added 4 commits June 8, 2026 21:05
## Dev Board Ticket

- https://dev.azure.com/TDEI-UW/TDEI/_workitems/edit/3726

## Changes
- Fixed OSW→OSM ID remapping so ways preserve references to the exact `ogr2osm` node objects instead of resolving by duplicate temporary numeric IDs.
- Prevented generated point-node IDs from being overwritten by OSW `_id` values, avoiding temporary node ID collisions before output.
- Added BUG-3726 regression fixtures with real San Diego coordinates.
- Added regression coverage to verify converted OSM ways keep local geometry and do not create cross-map spider lines.
- Bumped package version to `0.3.6` and updated the changelog.

## Testing
- `python -m unittest tests.unit_tests.test_osw2osm.test_osw2osm`
- `python -m unittest discover tests/unit_tests`
- Unit Tests workflow
- Coverage badge from the repo badges branch
@sujata-m
sujata-m requested review from MashB and susrisha June 10, 2026 09:08
@sujata-m
sujata-m merged commit 6164ff3 into main Jun 10, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants