search

godotengine / godot

Godot Engine – Multi-platform 2D and 3D game engine
https://godotengine.org
MIT License
91.22k stars 21.22k forks source link

[TRACKER] Render reorganisation #67600

Open BastiaanOlij opened 2 years ago

BastiaanOlij commented 2 years ago

Will be working on detailing this properly in the next few hours but after our rendering meeting we've decided to create a tracker for all the work done around the reorg and to list things still left to do.

This tracker only tracks issues in the master branch, not 3.x.

### Bugs
- [x] https://github.com/godotengine/godot/pull/65068
- [ ] https://github.com/godotengine/godot/issues/66857
### Enhancements
- [x] https://github.com/godotengine/godot/pull/59062
- [x] https://github.com/godotengine/godot/pull/59347
- [x] https://github.com/godotengine/godot/pull/59385
- [x] https://github.com/godotengine/godot/pull/59807
- [x] https://github.com/godotengine/godot/pull/59984
- [x] https://github.com/godotengine/godot/pull/60547
- [x] https://github.com/godotengine/godot/pull/60617
- [x] https://github.com/godotengine/godot/pull/61221
- [x] https://github.com/godotengine/godot/pull/62344
- [x] https://github.com/godotengine/godot/pull/62478
- [x] https://github.com/godotengine/godot/pull/63273
- [x] https://github.com/godotengine/godot/pull/63345
- [x] https://github.com/godotengine/godot/pull/63527
- [x] https://github.com/godotengine/godot/pull/63901
- [x] https://github.com/godotengine/godot/pull/65469
- [x] https://github.com/godotengine/godot/pull/65822
- [x] https://github.com/godotengine/godot/pull/70253
- [x] https://github.com/godotengine/godot/issues/64478
- [ ] https://github.com/godotengine/godot/issues/67602
- [ ] https://github.com/godotengine/godot/issues/67603
- [ ] https://github.com/godotengine/godot/issues/67604
- [ ] Final cleanup/round-off of render_scene_render around entry points and storage of settings
### Proposals
- [ ] https://github.com/godotengine/godot-proposals/issues/5616
BastiaanOlij commented 2 years ago

I've not created an issue for Final cleanup/round-off of rendering_scene_render around entry points and storage of settings, seemed overkill. This is mostly just a reminder to go through the rendering_scene_render_rd source and make sure we're happy with what is left in this class. There are probably still a few entry points that could move to rendering_method and call directly into storage/environment classes or something similar.

Especially around SDFGI there were a few I had some question marks with due to the way we expose SDFGI to Godot even if not supported.

We may also want to make renaming rendering_scene_cull and rendering_scene_render part of this?