Skip to content

Conversation

@jflann
Copy link
Member

@jflann jflann commented Mar 5, 2025

This PR re-implements the Street Addon Mod entirely in metarules. This implementation does not create any adjacency code, but rather depends on the NAM.dll to provide adjacency funtionality.

This PR also makes some big changes to the street network:

  • Eliminates support for zig-zag diagonal streets
  • Focuses on support for draggable diagonal streets
  • Reimplements many features via INRUL as opposed to RUL2
  • Makes it possible to draw the 3x3 smoothed street circle as well as the squared version
  • Restores the alternate form of the transition from orthogonal to diagonal (the "kinked bend")

This PR also incorporates a metarules implementation of the road roundabouts which was motivated by the necessity to change the roundabout implementation to accomodate draggable diagonal streets (#510)

@jflann jflann changed the title New SAM metarules implementation New SAM metarules implementation & street network changes Mar 5, 2025
jflann added 29 commits March 4, 2025 20:00
- remove unnecessary adjacency
- improve coverage
- add draggable implementation of diag s-curve
- add 3x3 street circle
- match DxO T metarules to INRUL-based implementation
- additional functions for curves
- improved code structure
- adds INRUL based diagonal connections
- eliminates diagonal connection RUL2
@jflann jflann changed the title New SAM metarules implementation & street network changes New SAM metarules implementation, street network changes, & road roundabout metarules Mar 20, 2025
memo33 added 3 commits March 22, 2025 20:59
…cal or Avenue networks

In particular, this avoids incorrect orientations such as `Avenue~SE`
and avoids flag `4`.

Also fixes a diagonal SAM × orth STR crossing ID.
@memo33 memo33 merged commit 8394e29 into staging Apr 28, 2025
2 checks passed
@memo33 memo33 deleted the meta-sam branch April 28, 2025 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants