Merge pull request #1584 from erenoku/navigation_cube

Don't require an aspect ratio when creating a new NavigationCubeRenderer
This commit is contained in:
Hanno Braun 2023-02-14 10:56:02 +01:00 committed by GitHub
commit 107a0c34c2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 12 deletions

View File

@ -1,7 +1,10 @@
use fj_math::Transform;
use wgpu::util::DeviceExt;
use super::model::{self, load_model, DrawModel, Model};
use super::{
model::{self, load_model, DrawModel, Model},
transform,
};
#[derive(Debug)]
pub struct NavigationCubeRenderer {
@ -19,7 +22,6 @@ impl NavigationCubeRenderer {
device: &wgpu::Device,
queue: &wgpu::Queue,
config: &wgpu::SurfaceConfiguration,
aspect_ratio: f64,
) -> Self {
let texture_bind_group_layout =
device.create_bind_group_layout(&wgpu::BindGroupLayoutDescriptor {
@ -48,13 +50,12 @@ impl NavigationCubeRenderer {
label: Some("texture_bind_group_layout"),
});
let mvp_matrix =
Self::get_mvp_matrix(Transform::identity(), aspect_ratio);
let mvp_matrix_buffer =
device.create_buffer_init(&wgpu::util::BufferInitDescriptor {
label: Some("Model Matrix Buffer"),
contents: bytemuck::cast_slice(&[mvp_matrix]),
contents: bytemuck::cast_slice(&[
transform::Transform::identity(),
]),
usage: wgpu::BufferUsages::UNIFORM
| wgpu::BufferUsages::COPY_DST,
});

View File

@ -186,12 +186,8 @@ impl Renderer {
let pipelines =
Pipelines::new(&device, &bind_group_layout, color_format);
let navigation_cube_renderer = NavigationCubeRenderer::new(
&device,
&queue,
&surface_config,
800.0 / 600.0,
);
let navigation_cube_renderer =
NavigationCubeRenderer::new(&device, &queue, &surface_config);
Ok(Self {
surface,