Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Streetplan support #503

Merged
merged 24 commits into from
Apr 11, 2024
Merged

Streetplan support #503

merged 24 commits into from
Apr 11, 2024

Conversation

Algorush
Copy link
Collaborator

@Algorush Algorush commented Mar 22, 2024

The first version is currently working with the test response from Streetplan: exampleJOSN.json. Buildings are not yet supported. The remaining segments mostly now work with default options.

What's next:

  • add other subtype options that can be displayed in 3DStreet.
  • add Buildings, setback.
  • add support for basic materials and markings. Currently, materials are selected only for bus-lane, bike-lane, streetcar, light-rail. But it would be nice to add other material options.
  • add support mixed segments in 3DStreet.
  • It would also be nice to add support for some other segment options in 3DStreet, which are quite easy to add: for example, empty roads, empty bus-lane, bike-lane,...

Also I'll make suggestions for Streetplan developer for JSON structure. For example it will be good to save units in JSON - feet/meters

Copy link

netlify bot commented Mar 22, 2024

Deploy Preview for 3dstreet-core-builds ready!

Name Link
🔨 Latest commit 2137cbd
🔍 Latest deploy log https://app.netlify.com/sites/3dstreet-core-builds/deploys/66182c30bd1b8f000844f8ff
😎 Deploy Preview https://deploy-preview-503--3dstreet-core-builds.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Algorush
Copy link
Collaborator Author

Added Setback segment support in last commits.
I forgot to say - streetplan.html is a test file with the streetplan-loader component. Currently, while cors on the Streetplan side is not configured, a test JSON file with the Streetplan scene is used, located in the src/streetplan/exampleJSON.json

- remove streetmix-loader dependency since streetplan-loader may be used instead
- add a bit more to our hacky matching string of streetplan to reduce unintended collisions, such as a streetmix street title
@kfarr kfarr merged commit a629f82 into main Apr 11, 2024
6 checks passed
@kfarr kfarr deleted the streetplan-support branch April 11, 2024 18:31
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.

None yet

2 participants