Tracker: 9.0 release

[ibgreen]

v9.0

Bugs

https://github.com/visgl/luma.gl/issues?q=is%3Aissue+is%3Aopen+milestone%3A9.0.0

Docs / Website

• api guide rendering.md - duplicate information at the end. fix(docs): documentation fixes per v9 tracker #1945
• engine AnimationLoop - contains a lot of old info
• engine RenderLoop -> AnimationLoopTemplate + incorrect info
• core Renderpipeline - contains GL constant references. @ibgreen fix(docs): documentation fixes per v9 tracker #1945
• core texture docs - likely contain incorrect info towards the end @ibgreen fix(docs): documentation fixes per v9 tracker #1945

Build

• Fix ESM plugins
• New ocular esbuild
• Restore script build chore: disable script builds #1533
• Fix babel-inline-webgl-constants Nice to have, not required for v9 - @donmccurdy chore(dev): Remove dev-modules/ from yarn workspace #1889 chore(ci): Disable babel inline webgl plugin #1870 babel-plugin-inline-webgl-constants v2.0.0-alpha.1 #1867 Fix babel-plugin-inline-webgl-constants #1859
• Fix debug module - @ibgreen
• Fix website - @ibgreen
• Cherry-pick circular dependencies PR Circular dependency in @luma.gl/webgl #1493 - @ibgreen
• .ts files - @ibgreen

API

• WebGL 1 Removal - @ibgreen -chore: Remove WebGL1 support #1933
• glTF tracker @felixpalmer Tracker: v9 glTF support #1847
• ShaderInputs @ibgreen
• Minimal TextureTransform port - @donmccurdy feat(engine): Transform → BufferTransform + TextureTransform #1896
• Minimal BufferTransform port - @donmccurdy [engine] Minimum Transform implementation #1879
• Refactor: Drop BufferWithAccessor subclass - @donmccurdy [webgl] Remove BufferWithAccessor #1864
• CommandEncoder,copyTextureToBuffer(): implement on webgl @donmccurdy
• Update all examples to use UniformBuffer - @ibgreen
• UniformBuffer support classes - @ibgreen
• @luma.gl/gltf - Restore glTF code and publish as separate module. - @felixpalmer
• Rename luma.gl/gltools to luma.gl/webgl-legacy - @ibgreen (API - OpenJS Steering Committee Feedback)

Tests

• Restore fp64 tests [engine] Minimum Transform implementation #1879 - @donmccurdy

Examples

• Simplify tutorial gltf example and write tutorial page? - @felixpalmer -
• Make WebGPU examples work again - @ibgreen - ShaderLayout refactor for deck.gl broke them
• Fix example layout issue (should not take full screen)
• Fix WebGL / WebGPU switcher in website @zbigg
• Fix example index page scrolling issue @zbigg
• New tutorial gltf example - @felixpalmer -

[donmccurdy]

Restore fp64 tests

I see these tests may have been restored in 5435295, is this still an issue?

[ibgreen]

I think that PR is just moving the shader module file structure around. Those tests are most likely still commented out.

[donmccurdy]

I see – so these blocks?

luma.gl/modules/shadertools/test/modules-webgl1/fp64/fp64-arithmetic-transform.spec.ts

Lines 77 to 79 in 42a1c87

	if (webgl2Device?.info.gpu === 'apple') {
	t.comment('apple GPU has precision issues')
	} else {

I'll take a look if no one is working on that already.

More links

• luma.gl/modules/shadertools/test/index.ts

  Lines 68 to 71 in 42a1c87

  	// Math modules
  	// TODO - these are breaking in test-browser but not in test-headless??
  	import './modules-webgl1/fp64/fp64-arithmetic-transform.spec';
  	import './modules-webgl1/fp64/fp64-utils.spec';

• modules/shadertools/test/modules-webgl1/fp64/fp64-utils.spec.ts
• modules/shadertools/test/modules-webgl1/fp64/fp64-arithmetic-transform.spec.ts

On closer look, those tests depend on transform feedback, which is still an open task. I'll start chipping away at Ib's suggestion, "Most of the tests should be moved from webgl into core-tests module where they can run on both webgl and later webgpu backends", instead.

[donmccurdy]

CommandEncoder: encapsulate webgl readPixel()`

@ibgreen I've restored tests for copyTextureToBuffer in #1885, that seems to be working, is anything else required for this task?

[ibgreen]

CommandEncoder: encapsulate webgl readPixel()`

@ibgreen I've restored tests for copyTextureToBuffer in #1885, that seems to be working, is anything else required for this task?

Ultimately the reason we did this is to make applications perform texture reads through a portable API. Right now apps (and deck.gl) have to import webgl module. See e.g. https://github.com/visgl/luma.gl/blob/master/examples/showcase/instancing/app.ts#L6

You could try and see if it is not too hard to replace all use of readPixels, but then again we don't necessarily need to solve this completely in 9.0, we could come back to this in 9.1.

Still, it is good to have this enabler in place!