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

Guard against NaNs in shader #7018

Merged
merged 3 commits into from
Apr 26, 2023
Merged

Guard against NaNs in shader #7018

merged 3 commits into from
Apr 26, 2023

Conversation

philippotto
Copy link
Member

@philippotto philippotto commented Apr 25, 2023

When deciding whether to use a performance optimization in the shader, some systems produce NaN which leads to black blocks.

URL of deployed dev instance (used for testing):

  • https://___.webknossos.xyz

Steps to test:

  • I used the adapted code on an affected system (Windows 10, Lenovo laptop, Intel i7-4710MQ, 8GB RAM, Intel’s HD graphics 4600 and Nvidia GeForce GT 730M).

Issues:


(Please delete unneeded items, merge only when none are left open)

@philippotto philippotto self-assigned this Apr 25, 2023
@philippotto philippotto requested a review from fm3 April 26, 2023 08:53
@philippotto philippotto enabled auto-merge (squash) April 26, 2023 09:04
@philippotto philippotto merged commit 69ce101 into master Apr 26, 2023
@philippotto philippotto deleted the nan-guard branch April 26, 2023 09:11
hotzenklotz added a commit that referenced this pull request Apr 28, 2023
…ove_wkconnect

* 'master' of github.com:scalableminds/webknossos:
  Update docker compose commands + dev install readme (#7002)
  Add segment groups (#6966)
  Add screenshot nightly test for wkorg (#7030)
  Workaround for WebGL crash for datasets with many segmentation layers (#6995)
  Fix download of public annotation, include access ctx in user cache key (#7025)
  Fix that changing a segment color could lead to a crash (#7000)
  Add more error chaining to annotation download (#7023)
  Guard against NaNs in shader (#7018)
  Store editable mappings in multiple fossildb columns+keys (#6903)
  Context action to move tree to group (#7005)
  Release 23.05.0 (#7014)
  Remove vault cache when reloading dataset (#7007)
  Fix viewing of public datasets (#7010)
  Update screenshots scalebar positioning (#7003)
  Update team members (#6999)
philippotto added a commit that referenced this pull request May 2, 2023
* guard against nan when deciding whether to use performance optimization in shader

* update changelog
philippotto added a commit that referenced this pull request May 3, 2023
* Release 23.05.0

* Guard against NaNs in shader (#7018)

* guard against nan when deciding whether to use performance optimization in shader

* update changelog

* Workaround for WebGL crash for datasets with many segmentation layers (#6995)

* reduce shader complexity when having multiple segmentation layers (shader needs to be compiled more often then, though)

* fix tests

* clean up

* update changelog

* Release 23.05.1

* fix incorrect merge
philippotto added a commit that referenced this pull request May 8, 2023
* Release 23.05.0

* Guard against NaNs in shader (#7018)

* guard against nan when deciding whether to use performance optimization in shader

* update changelog

* Workaround for WebGL crash for datasets with many segmentation layers (#6995)

* reduce shader complexity when having multiple segmentation layers (shader needs to be compiled more often then, though)

* fix tests

* clean up

* update changelog

* Release 23.05.1

* Fix access check in time tracking controller (#7055)

* Release 23.05.2

---------

Co-authored-by: Florian M <[email protected]>
hotzenklotz added a commit that referenced this pull request May 17, 2023
…ty-list-drawings

* 'master' of github.com:scalableminds/webknossos: (25 commits)
  Fix issues with styling in dark mode on login page (#7052)
  Fix nightly by setting missing token (#7048)
  Release 23.05.1 (#7042)
  DRY types in update_actions.ts (#7036)
  Remove some spammy logging from backend (#7039)
  Use zarr string fill values (#7017)
  Fix voxel offset for Neuroglancer Precomputed datasets (#7019)
  Log when user is activated (#7027)
  Fix exception in applying UpdateTreeGroupVisibility skeleton action (#7037)
  Fix organization storage layouting (#7034)
  Update docker compose commands + dev install readme (#7002)
  Add segment groups (#6966)
  Add screenshot nightly test for wkorg (#7030)
  Workaround for WebGL crash for datasets with many segmentation layers (#6995)
  Fix download of public annotation, include access ctx in user cache key (#7025)
  Fix that changing a segment color could lead to a crash (#7000)
  Add more error chaining to annotation download (#7023)
  Guard against NaNs in shader (#7018)
  Store editable mappings in multiple fossildb columns+keys (#6903)
  Context action to move tree to group (#7005)
  ...
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.

Rendering problems on certain machines
2 participants