-
Notifications
You must be signed in to change notification settings - Fork 1
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
Pointcloud raytracing #41
Conversation
@FlorianK13 @MartGro , what are your opinions? (regarding how exactly to integrate / UX wise etc.) |
|
Still get the same error: In Edge the error message is: Aborted(CompileError: WebAssembly.instantiate(): expected magic word 00 61 73 6d, found 2e 2e 2f 2e @+0) |
Running on windows? It might be that you do not get the symbolic link. And/or you have to re-run |
I get a laz-perf.wasm in static/js folder. I also rerun npm install. Yes It is on windows. The error persists, still the wasm validation error. @MartGro Is it running on your laptop? |
6cee60f
to
874ad21
Compare
We resolved it now. On windows one has to manually copy the symlinked file The latest updates integrate the translations and make it possible to opt out of laser data. Also laser data is loaded now from the LVA Bavaria proxied via our website - so all of Bavaria is usable. From my side this would now be ready to merge. Any further comments? |
Restarting the simulation via the arrows results in the error: 'Uncaught (in promise) TypeError: coordinates must be finite numbers' |
I cannot reproduce. Did you pull the latest commits? Please provide more detailed instructions on the error. |
Sorry, forgot to pull the commits 🙃 |
Is it right / intended that the raytracing is only done with the points? At least I get this intention when I look at 'Rathaus Augsburg' and show the pointcloud. Then, at the right side of the building there is no real shadowing even though the next building is very close (The wall of the next building only has no points from the pointcloud) |
Additionally repair the enableLaserPoint variable
First draft for integration point cloud data. Definitely not mergable, but as a first point of discussion on how to progress.