Merge weBIGeo#234
Merged
Merged
Conversation
- uv/position/weight calculations should be finally correct now
- for experimental implementation of multi-flow models using a sequence of compute shader calls
Note: Seems to be buggy for overlay texture
- convert our single-direction trajectory model to a monte-carlo method by randomly offsetting the sampled normal each step for simulating divergence/"spread" of the avalanche - still WIP, need to expose some parameters as uniforms esp. sample count and offset radius
- dispatch more workgroups instead of looping over samples in shader
- eliminates annoying white trajectories per default
What we did in the meeting.
- this needs to be fixed in the model
* fix: ktx and ktx_read build directory * moves. binary docs and adds stub for webgpu-app.md * add: Readme based on upstream * add: round shape for library * add: show only top level dependencies in graph * add: diagram to explain webgpu_app * add: doc for webgpu_engine * add: minify rendering pipeline * add: documentation for webgpu_base * add: links to Readme * add: simplification for webgpu_app_dev * add: test cmake targets to documentation
# Conflicts: # .gitignore # CMakeLists.txt # README.md
Develop Merge
* add: removes hardcoded avalanche disclaimer * add: removes all <format> uses (no gcc support) * add: removes failing roundtrip test * fix: case insensitive QDebug
removes: gpx_tracks and disables webgpu targets for ci pipelines
fix: fixes cmake presets and adds alp-base
…arder to select on ci runs
…ths yet, but one step at a time..
* add: status and fadeout for spinner * add: more contrast logo * add: analyze_wasm script and puts nucleus ressources behind proper flag * add: ImGuiFileDialog only for native build
Minor shell gui changes
* Enable WebGPU app artifacts in CI deploy * Guard Windows WebGPU app library pragma * Fix WebGPU vertex format assertion for GCC * Treat WebGPU timestamp query as optional * Install Xlib XCB headers for Linux CI * Retry WebGPU device without timestamp queries * Load WebGPU app script once * Suppress WebGPU app asyncify unwind rejection * Pin Windows CI aqtinstall source * Pin Windows CI aqtinstall source * ignore++ * rename binary docs to project reports
adam-ce
commented
Jun 30, 2026
adam-ce
commented
Jun 30, 2026
Member
Author
There was a problem hiding this comment.
i'm not against providing a cmake preset, but this one should be cleaned up.
there are hard coded paths, alp-base sounds strange in the context of the upstream repo, generally, there is a ton of targets.
imo:
- remove the release stuff, and let CI build that (you can download the artefacts).
- make targets like webgpu_* and webgl_*
- remove all hardcoded paths (they will break at some point), and rely on the environment. if that doesn't work, instruct the user on how to specify it locally.
Contributor
There was a problem hiding this comment.
I use the cmake presets frequently. Renaming them is not an issue but removing targets is not really in my interest. I honestly think they would be a good fit for the ci pipelines aswell (such that they just need to call the proper preset and install the files) But thats just how I would do it.
I also extracted most of the hardcoded paths and now set it localy, but I couldnt find a proper replacement for the toolchainFile, so this path is still in there.
If you have a big opinion against it I'm happy for alternative proposals. What I can offer you is a seperate pr from a branch with the cmakepresets file deleted.
Member
Author
There was a problem hiding this comment.
alright. then just rename everything you need to webgpu*, and i might add webgl presets at some point :)
* reactivates report_global_cursor_position * removes unused stb_image libraries * removes max_n_tiles and replaces wendelins asserts with qassert * removes GpuTileId and uses nucleus packing functions instead * fix: line endings linux style and force for all with .gitattributes * add: setupKTX function for main CMakeLists * bump required min qt version * add: restore README links * cmakepresets cleanup * fix: wasm serve script
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.