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

Renderer service #197

Merged
merged 45 commits into from
Dec 14, 2023
Merged

Renderer service #197

merged 45 commits into from
Dec 14, 2023

Conversation

marceline-cramer
Copy link
Collaborator

@marceline-cramer marceline-cramer commented Nov 5, 2023

Creates a new hearth-renderer crate and exposes as much as is feasible of the rend3 and rend3-routine APIs to guests through a message protocol.

  • add initial renderer protocol to hearth-types
  • add initial hearth-renderer crate
  • add JsonAssetLoader helper trait and blanket AssetLoader impl
  • implement Rend3Command for modifying global rend3 state
  • load skyboxes
  • change ambient lighting
  • define stub asset loaders for materials, meshes, textures, and cube textures
  • define protocol for some minimal set of texture data and implement loading
  • spawn objects and handle object updates
  • define protocol for full mesh data and implement loading
  • somehow allow asset loaders to load from other asset loaders
  • remove hardcoded directional light
  • spawn directional light processes and handle light updates
  • use export_to for cleaner instance spawning
  • base64-encode bulky data
  • use the time PR's spawn utils for instances
  • validate every asset type

Follow-up:

  • improve compilation speed of asset loaders?
  • define protocol for full rend3 support of texture data and implement loading
  • generate texture mips?
  • support batched object spawns
  • define protocol for full material processing and implement loading
  • point lights
  • write a kindling-host API

@marceline-cramer marceline-cramer marked this pull request as ready for review December 14, 2023 01:45
Copy link
Collaborator

@airidaceae airidaceae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! I'm going to approve this because I would be happy merging it but I think that one helper function would be nice to have beforehand.

plugins/renderer/src/lib.rs Outdated Show resolved Hide resolved
@marceline-cramer marceline-cramer merged commit a35cbb0 into main Dec 14, 2023
10 checks passed
@marceline-cramer marceline-cramer deleted the renderer-service branch December 14, 2023 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants