Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions Cargo.toml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
[package]
edition = "2021"
name = "bevy_polyline"
version = "0.12.0"
version = "0.13.0"
description = "Polyline Rendering for Bevy"
license = "MIT OR Apache-2.0"
repository = "https://github.com/ForesightMiningSoftwareCorporation/bevy_polyline"
Expand All @@ -18,23 +18,24 @@ authors = [

[dependencies]
bitflags = "2.3"
bevy = { version = "0.16", default-features = false, features = [
bevy = { version = "0.17", default-features = false, features = [
"bevy_core_pipeline",
"bevy_render",
"bevy_asset",
] }
bevy_post_process = "0.17"
bytemuck = "1.16.1"

[dev-dependencies]
lazy_static = "1.4.0"
rand = "0.8.4"
ringbuffer = "0.15"
bevy = { version = "0.16", default-features = false, features = [
bevy = { version = "0.17", default-features = false, features = [
"bevy_window",
"bevy_winit",
"bevy_pbr",
"x11",
"tonemapping_luts",
"ktx2",
"zstd",
"zstd_rust",
] }
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ We intend to track the `main` branch of Bevy. PRs supporting this are welcome!

| bevy | bevy_polyline |
| ---- | ------------- |
| 0.17 | 0.13 |
| 0.16 | 0.12 |
| 0.15 | 0.11 |
| 0.14 | 0.10 |
Expand Down
8 changes: 3 additions & 5 deletions examples/minimal.rs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
use bevy::{color::palettes::css::RED, prelude::*};
use bevy::{color::palettes::css::RED, prelude::*, render::view::Hdr};
use bevy_polyline::prelude::*;

fn main() {
Expand Down Expand Up @@ -30,10 +30,8 @@ fn setup(
// camera
commands.spawn((
Camera3d::default(),
Camera {
hdr: true,
..default()
},
Camera::default(),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think you could just remove this line now since Camera3d requires Camera.

Hdr,
Msaa::Sample4,
Transform::from_xyz(0.0, 0.0, 5.0).looking_at(Vec3::ZERO, Vec3::Y),
));
Expand Down
10 changes: 5 additions & 5 deletions examples/nbody.rs
Original file line number Diff line number Diff line change
@@ -1,12 +1,14 @@
use std::f32::consts::PI;

use bevy::{
core_pipeline::{bloom::Bloom, tonemapping::Tonemapping},
core_pipeline::tonemapping::Tonemapping,
diagnostic::{FrameTimeDiagnosticsPlugin, LogDiagnosticsPlugin},
math::Vec3A,
prelude::*,
render::view::Hdr,
};
use bevy_polyline::prelude::*;
use bevy_post_process::bloom::Bloom;

use lazy_static::*;
use rand::{prelude::*, Rng};
Expand Down Expand Up @@ -79,10 +81,8 @@ fn setup(
// camera
commands.spawn((
Camera3d::default(),
Camera {
hdr: true,
..default()
},
Camera::default(),
Hdr,
Msaa::Sample4,
Tonemapping::TonyMcMapface,
Bloom::default(),
Expand Down
8 changes: 3 additions & 5 deletions examples/perspective.rs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
use bevy::{color::palettes::css::RED, prelude::*};
use bevy::{color::palettes::css::RED, prelude::*, render::view::Hdr};
use bevy_polyline::prelude::*;

fn main() {
Expand Down Expand Up @@ -33,10 +33,8 @@ fn setup(
Camera3d::default(),
Msaa::Sample4,
Transform::from_xyz(0.0, 0.0, 2.0).looking_at(Vec3::ZERO, Vec3::Y),
Camera {
hdr: true,
..default()
},
Camera::default(),
Hdr,
));
}

Expand Down
4 changes: 2 additions & 2 deletions src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
#![allow(clippy::too_many_arguments)]

use bevy::{
asset::{load_internal_asset, weak_handle},
asset::{load_internal_asset, uuid_handle},
prelude::*,
};
use material::PolylineMaterialPlugin;
Expand All @@ -18,7 +18,7 @@ pub mod prelude {
}
pub struct PolylinePlugin;

pub const SHADER_HANDLE: Handle<Shader> = weak_handle!("b180bfe9-10c8-48fe-b27a-dfa41436d7d0");
pub const SHADER_HANDLE: Handle<Shader> = uuid_handle!("b180bfe9-10c8-48fe-b27a-dfa41436d7d0");

impl Plugin for PolylinePlugin {
fn build(&self, app: &mut bevy::prelude::App) {
Expand Down
18 changes: 11 additions & 7 deletions src/material.rs
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ use bevy::{
render_resource::{binding_types::uniform_buffer, *},
renderer::{RenderDevice, RenderQueue},
view::{ExtractedView, RenderVisibleEntities, ViewUniformOffset},
Render, RenderApp, RenderSet,
Render, RenderApp, RenderSystems,
},
};
use std::fmt::Debug;
Expand Down Expand Up @@ -124,6 +124,7 @@ impl RenderAsset for GpuPolylineMaterial {
polyline_material: Self::SourceAsset,
_: AssetId<Self::SourceAsset>,
(device, queue, polyline_pipeline): &mut bevy::ecs::system::SystemParamItem<Self::Param>,
_: Option<&Self>,
) -> Result<Self, PrepareAssetError<Self::SourceAsset>> {
let value = PolylineMaterialUniform {
width: polyline_material.width,
Expand Down Expand Up @@ -178,7 +179,10 @@ impl Plugin for PolylineMaterialPlugin {
.add_render_command::<AlphaMask3d, DrawPolylineMaterial>()
.init_resource::<PolylineMaterialPipeline>()
.init_resource::<SpecializedRenderPipelines<PolylineMaterialPipeline>>()
.add_systems(Render, queue_material_polylines.in_set(RenderSet::Queue));
.add_systems(
Render,
queue_material_polylines.in_set(RenderSystems::Queue),
);
}
}
}
Expand Down Expand Up @@ -237,8 +241,8 @@ impl<const I: usize, P: PhaseItem> RenderCommand<P> for SetPolylineViewBindGroup
#[inline]
fn render<'w>(
_item: &P,
(view_uniform, mesh_view_bind_group): ROQueryItem<'w, Self::ViewQuery>,
_entity: Option<ROQueryItem<'w, Self::ItemQuery>>,
(view_uniform, mesh_view_bind_group): ROQueryItem<'w, '_, Self::ViewQuery>,
_entity: Option<ROQueryItem<'w, '_, Self::ItemQuery>>,
_param: SystemParamItem<'w, '_, Self::Param>,
pass: &mut TrackedRenderPass<'w>,
) -> RenderCommandResult {
Expand All @@ -255,8 +259,8 @@ impl<const I: usize, P: PhaseItem> RenderCommand<P> for SetMaterialBindGroup<I>

fn render<'w>(
_item: &P,
_view: ROQueryItem<'w, Self::ViewQuery>,
material_handle: Option<ROQueryItem<'w, Self::ItemQuery>>,
_view: ROQueryItem<'w, '_, Self::ViewQuery>,
material_handle: Option<ROQueryItem<'w, '_, Self::ItemQuery>>,
materials: SystemParamItem<'w, '_, Self::Param>,
pass: &mut TrackedRenderPass<'w>,
) -> RenderCommandResult {
Expand Down Expand Up @@ -297,7 +301,7 @@ pub fn queue_material_polylines(
.id::<DrawPolylineMaterial>();

for (view, visible_entities, msaa) in &views {
let inverse_view_matrix = view.world_from_view.compute_matrix().inverse();
let inverse_view_matrix = view.world_from_view.to_matrix().inverse();
let inverse_view_row_2 = inverse_view_matrix.row(2);

let mut polyline_key = PolylinePipelineKey::from_msaa_samples(msaa.samples());
Expand Down
29 changes: 16 additions & 13 deletions src/polyline.rs
Original file line number Diff line number Diff line change
@@ -1,12 +1,14 @@
use crate::material::PolylineMaterialHandle;
use bevy::{
camera::{self, visibility::VisibilityClass},
ecs::{
query::ROQueryItem,
system::{
lifetimeless::{Read, SRes},
SystemParamItem,
},
},
mesh::VertexBufferLayout,
prelude::*,
reflect::TypePath,
render::{
Expand All @@ -16,8 +18,8 @@ use bevy::{
render_resource::{binding_types::uniform_buffer, *},
renderer::RenderDevice,
sync_world::{RenderEntity, SyncToRenderWorld},
view::{self, ViewUniform, ViewUniforms, VisibilityClass},
Extract, Render, RenderApp, RenderSet,
view::{ViewUniform, ViewUniforms},
Extract, Render, RenderApp, RenderSystems,
},
};

Expand All @@ -43,8 +45,8 @@ impl Plugin for PolylineRenderPlugin {
.add_systems(
Render,
(
prepare_polyline_bind_group.in_set(RenderSet::PrepareBindGroups),
prepare_polyline_view_bind_groups.in_set(RenderSet::PrepareBindGroups),
prepare_polyline_bind_group.in_set(RenderSystems::PrepareBindGroups),
prepare_polyline_view_bind_groups.in_set(RenderSystems::PrepareBindGroups),
),
);
}
Expand All @@ -70,7 +72,7 @@ pub struct Polyline {

#[derive(Debug, Clone, Default, Component)]
#[require(SyncToRenderWorld, VisibilityClass)]
#[component(on_add = view::add_visibility_class::<PolylineHandle>)]
#[component(on_add = camera::visibility::add_visibility_class::<PolylineHandle>)]
pub struct PolylineHandle(pub Handle<Polyline>);

impl RenderAsset for GpuPolyline {
Expand All @@ -82,6 +84,7 @@ impl RenderAsset for GpuPolyline {
polyline: Self::SourceAsset,
_: AssetId<Self::SourceAsset>,
render_device: &mut bevy::ecs::system::SystemParamItem<Self::Param>,
_: Option<&Self>,
) -> Result<Self, PrepareAssetError<Self::SourceAsset>> {
let vertex_buffer_data = bytemuck::cast_slice(polyline.vertices.as_slice());
let vertex_buffer = render_device.create_buffer_with_data(&BufferInitDescriptor {
Expand Down Expand Up @@ -127,11 +130,11 @@ pub fn extract_polylines(
if !inherited_visibility.get() || !view_visibility.get() {
continue;
}
let transform = transform.compute_matrix();
let transform = transform.to_matrix();
values.push((
entity,
(
PolylineHandle(handle.0.clone_weak()),
PolylineHandle(handle.0.clone()),
PolylineUniform { transform },
),
));
Expand Down Expand Up @@ -221,7 +224,7 @@ impl SpecializedRenderPipeline for PolylinePipeline {
RenderPipelineDescriptor {
vertex: VertexState {
shader: self.shader.clone(),
entry_point: "vertex".into(),
entry_point: Some("vertex".into()),
shader_defs: shader_defs.clone(),
buffers: vec![vertex_layout.clone(), {
vertex_layout.attributes[0].shader_location = 1;
Expand All @@ -231,7 +234,7 @@ impl SpecializedRenderPipeline for PolylinePipeline {
fragment: Some(FragmentState {
shader: self.shader.clone(),
shader_defs,
entry_point: "fragment".into(),
entry_point: Some("fragment".into()),
targets: vec![Some(ColorTargetState {
format,
blend,
Expand Down Expand Up @@ -370,8 +373,8 @@ impl<const I: usize, P: PhaseItem> RenderCommand<P> for SetPolylineBindGroup<I>
#[inline]
fn render<'w>(
_item: &P,
_view: ROQueryItem<'w, Self::ViewQuery>,
polyline_index: Option<ROQueryItem<'w, Self::ItemQuery>>,
_view: ROQueryItem<'w, '_, Self::ViewQuery>,
polyline_index: Option<ROQueryItem<'w, '_, Self::ItemQuery>>,
bind_group: SystemParamItem<'w, '_, Self::Param>,
pass: &mut TrackedRenderPass<'w>,
) -> RenderCommandResult {
Expand All @@ -392,8 +395,8 @@ impl<P: PhaseItem> RenderCommand<P> for DrawPolyline {
#[inline]
fn render<'w>(
_item: &P,
_view: ROQueryItem<'w, Self::ViewQuery>,
pl_handle: Option<ROQueryItem<'w, Self::ItemQuery>>,
_view: ROQueryItem<'w, '_, Self::ViewQuery>,
pl_handle: Option<ROQueryItem<'w, '_, Self::ItemQuery>>,
polylines: SystemParamItem<'w, '_, Self::Param>,
pass: &mut TrackedRenderPass<'w>,
) -> RenderCommandResult {
Expand Down
Loading