Skip to content

Conversation

@sujata-m
Copy link
Contributor

Dev Board Ticket

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

Changes

  • Restored custom field support across schemas (CustomNode/Edge/Zone) while aligning required fields with original behavior.
  • Relaxed edges/zones global required lists to let custom features omit highway (kept in branch-specific schemas).
  • Enhanced 0.2 compatibility guard: allow ext: on nodes, block custom content per dataset with specific messages (e.g., Custom Edge/Point/Polygon), and reject non-point ext:.
  • Added schema parity tests and fixtures for custom feature branches and 0.2 vs 0.3 behavior, including guard reason checks.
  • Improved diff tooling/reporting and parity fixtures to cover custom scenarios.

Testing

  • Added 14 new test cases (all in tests/test_schema_parity.py) and 30 new fixture files under tests/schema/fixtures (covering nodes/edges/lines/points/polygons/zones).

sujata-m and others added 2 commits December 15, 2025 16:33
## Dev Board Ticket

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

## Changes
- Restored custom field support across schemas (CustomNode/Edge/Zone) while aligning required fields with original behavior.
- Relaxed edges/zones global required lists to let custom features omit `highway` (kept in branch-specific schemas).
- Enhanced 0.2 compatibility guard: allow `ext:` on nodes, block custom content per dataset with specific messages (e.g., Custom Edge/Point/Polygon), and reject non-point `ext:`.
- Added schema parity tests and fixtures for custom feature branches and 0.2 vs 0.3 behavior, including guard reason checks.
- Improved diff tooling/reporting and parity fixtures to cover custom scenarios.

## Testing
- Added 14 new test cases (all in tests/test_schema_parity.py) and 30 new fixture files under tests/schema/fixtures (covering nodes/edges/lines/points/polygons/zones).
@sujata-m sujata-m requested review from MashB and susrisha December 15, 2025 12:32
@sujata-m sujata-m merged commit c84ea45 into main Dec 15, 2025
7 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.

3 participants