render: remove IDs for everything, use only labels to identify things
ci/woodpecker/pr/debug Pipeline failed Details
ci/woodpecker/pr/release Pipeline failed Details

This commit is contained in:
SeanOMik 2024-06-11 20:59:55 -04:00
parent 28b9604189
commit 6d7932f6a5
Signed by: SeanOMik
GPG Key ID: FEC9E2FC15235964
8 changed files with 150 additions and 211 deletions

View File

@ -1,9 +1,8 @@
mod node; mod node;
use std::{ use std::{
cell::RefCell, collections::{HashMap, VecDeque}, fmt::Debug, hash::Hash, rc::Rc, sync::Arc cell::{Ref, RefCell}, collections::{HashMap, VecDeque}, fmt::Debug, hash::Hash, rc::Rc, sync::Arc
}; };
use itertools::Itertools;
use lyra_ecs::World; use lyra_ecs::World;
pub use node::*; pub use node::*;
@ -34,6 +33,8 @@ pub trait RenderGraphLabel: Debug + 'static {
} }
} }
pub struct RenderGraphHash(u64);
#[derive(Clone)] #[derive(Clone)]
pub struct RenderGraphLabelValue(Rc<dyn RenderGraphLabel>); pub struct RenderGraphLabelValue(Rc<dyn RenderGraphLabel>);
@ -77,9 +78,10 @@ impl Eq for RenderGraphLabelValue {}
struct PassEntry { struct PassEntry {
inner: Arc<RefCell<dyn Node>>, inner: Arc<RefCell<dyn Node>>,
desc: Arc<NodeDesc>, desc: Rc<RefCell<NodeDesc>>,
/// The index of the pass in the execution graph /// The index of the pass in the execution graph
graph_index: petgraph::matrix_graph::NodeIndex<usize>, graph_index: petgraph::matrix_graph::NodeIndex<usize>,
pipeline: Rc<RefCell<Option<PipelineResource>>>,
} }
pub struct BindGroupEntry { pub struct BindGroupEntry {
@ -116,20 +118,19 @@ pub struct RenderTarget {
pub struct RenderGraph { pub struct RenderGraph {
device: Rc<wgpu::Device>, device: Rc<wgpu::Device>,
queue: Rc<wgpu::Queue>, queue: Rc<wgpu::Queue>,
slots: FxHashMap<u64, ResourcedSlot>, slots: FxHashMap<RenderGraphLabelValue, ResourcedSlot>,
/// HashMap used to lookup the slot id using the label's hash /// HashMap used to lookup the slot id using the label's hash
slot_label_lookup: FxHashMap<RenderGraphLabelValue, u64>, //slot_label_lookup: FxHashMap<RenderGraphLabelValue, u64>,
passes: FxHashMap<u64, PassEntry>, passes: FxHashMap<RenderGraphLabelValue, PassEntry>,
// TODO: Use a SlotMap // TODO: Use a SlotMap
bind_groups: FxHashMap<u64, BindGroupEntry>, bind_groups: FxHashMap<RenderGraphLabelValue, BindGroupEntry>,
/// HashMap used to lookup the bind group id using the label's hash /// HashMap used to lookup the bind group id using the label's hash
bind_group_names: FxHashMap<RenderGraphLabelValue, u64>, //bind_group_names: FxHashMap<RenderGraphLabelValue, u64>,
// TODO: make pipelines a `type` parameter in RenderPasses, // TODO: make pipelines a `type` parameter in RenderPasses,
// then the pipelines can be retrieved via TypeId to the pass. // then the pipelines can be retrieved via TypeId to the pass.
pipelines: FxHashMap<u64, PipelineResource>, //pipelines: FxHashMap<u64, PipelineResource>,
current_id: u64,
/// A directed graph describing the execution path of the RenderGraph /// A directed graph describing the execution path of the RenderGraph
execution_graph: petgraph::matrix_graph::DiMatrix<u64, (), Option<()>, usize>, execution_graph: petgraph::matrix_graph::DiMatrix<RenderGraphLabelValue, (), Option<()>, usize>,
} }
impl RenderGraph { impl RenderGraph {
@ -138,12 +139,8 @@ impl RenderGraph {
device, device,
queue, queue,
slots: Default::default(), slots: Default::default(),
slot_label_lookup: Default::default(),
passes: Default::default(), passes: Default::default(),
bind_groups: Default::default(), bind_groups: Default::default(),
bind_group_names: Default::default(),
pipelines: Default::default(),
current_id: 1,
execution_graph: Default::default(), execution_graph: Default::default(),
} }
} }
@ -152,19 +149,6 @@ impl RenderGraph {
&*self.device &*self.device
} }
pub fn next_id(&mut self) -> u64 {
self.current_id += 1;
self.current_id
}
pub(crate) fn slot_id_rc(&self, label: &RenderGraphLabelValue) -> Option<u64> {
self.slot_label_lookup.get(&label.clone().into()).cloned()
}
pub fn slot_id(&self, label: &dyn RenderGraphLabel) -> Option<u64> {
self.slot_label_lookup.get(&label.rc_clone().into()).cloned()
}
/// Add a [`Node`] to the RenderGraph. /// Add a [`Node`] to the RenderGraph.
/// ///
/// When the node is added, its [`Node::desc`] method will be executed. /// When the node is added, its [`Node::desc`] method will be executed.
@ -176,34 +160,35 @@ impl RenderGraph {
/// not change. The IDs of output slots do stay the same. /// not change. The IDs of output slots do stay the same.
/// 3. Ensuring that no two slots share the same ID when the names do not match. /// 3. Ensuring that no two slots share the same ID when the names do not match.
#[instrument(skip(self, pass), level = "debug")] #[instrument(skip(self, pass), level = "debug")]
pub fn add_pass<P: Node>(&mut self, mut pass: P) { pub fn add_pass<P: Node>(&mut self, label: impl RenderGraphLabel, mut pass: P) {
let mut desc = pass.desc(self); let mut desc = pass.desc(self);
// collect all the slots of the pass // collect all the slots of the pass
for slot in &mut desc.slots { for slot in &mut desc.slots {
if let Some((id, other)) = self if let Some(other) = self
.slot_label_lookup .slots
.get(&slot.label) .get_mut(&slot.label)
.and_then(|id| self.slots.get_mut(id).map(|s| (id, s))) //.map(|s| (id, s))
//.and_then(|id| self.slots.get_mut(id).map(|s| (id, s)))
{ {
debug_assert_eq!( debug_assert_eq!(
slot.ty, other.ty, slot.ty, other.ty,
"slot {:?} in pass {:?} does not match existing slot of same name", "slot {:?} in pass {:?} does not match existing slot of same name",
slot.label, desc.label slot.label, label
); );
trace!( /* trace!(
"Found existing slot for {:?}, changing id to {}", "Found existing slot for {:?}, changing id to {}",
slot.label, slot.label,
id id
); ); */
// if there is a slot of the same name // if there is a slot of the same name
slot.id = *id; //slot.id = *id;
} else { } else {
debug_assert!(!self.slots.contains_key(&slot.id), debug_assert!(!self.slots.contains_key(&slot.label),
"Reuse of id detected in render graph! Pass: {:?}, slot: {:?}", "Reuse of id detected in render graph! Pass: {:?}, slot: {:?}",
desc.label, slot.label, label, slot.label,
); );
let res_slot = ResourcedSlot { let res_slot = ResourcedSlot {
@ -212,33 +197,29 @@ impl RenderGraph {
value: slot.value.clone().unwrap_or(SlotValue::None), value: slot.value.clone().unwrap_or(SlotValue::None),
}; };
self.slots.insert(slot.id, res_slot); self.slots.insert(slot.label.clone(), res_slot);
self.slot_label_lookup.insert(slot.label.clone(), slot.id);
} }
} }
// get clones of the bind groups and layouts // get clones of the bind groups and layouts
for (label, bg, bgl) in &desc.bind_groups { for (label, bg, bgl) in &desc.bind_groups {
let bg_id = self.next_id(); self.bind_groups.insert(label.clone(), BindGroupEntry {
self.bind_groups.insert(
bg_id,
BindGroupEntry {
label: label.clone(), label: label.clone(),
bg: bg.clone(), bg: bg.clone(),
layout: bgl.clone(), layout: bgl.clone(),
}, });
);
self.bind_group_names.insert(label.clone().into(), bg_id);
} }
let index = self.execution_graph.add_node(desc.id); let label: RenderGraphLabelValue = label.into();
let index = self.execution_graph.add_node(label.clone());
self.passes.insert( self.passes.insert(
desc.id, label,
PassEntry { PassEntry {
inner: Arc::new(RefCell::new(pass)), inner: Arc::new(RefCell::new(pass)),
desc: Arc::new(desc), desc: Rc::new(RefCell::new(desc)),
graph_index: index, graph_index: index,
pipeline: Rc::new(RefCell::new(None)),
}, },
); );
} }
@ -250,9 +231,10 @@ impl RenderGraph {
#[instrument(skip(self, device))] #[instrument(skip(self, device))]
pub fn setup(&mut self, device: &wgpu::Device) { pub fn setup(&mut self, device: &wgpu::Device) {
// For all passes, create their pipelines // For all passes, create their pipelines
for pass in self.passes.values() { for pass in self.passes.values_mut() {
if let Some(pipeline_desc) = &pass.desc.pipeline_desc { let desc = (*pass.desc).borrow();
let pipeline = match pass.desc.ty { if let Some(pipeline_desc) = &desc.pipeline_desc {
let pipeline = match desc.ty {
NodeType::Render => Pipeline::Render(RenderPipeline::create( NodeType::Render => Pipeline::Render(RenderPipeline::create(
device, device,
pipeline_desc pipeline_desc
@ -270,11 +252,14 @@ impl RenderGraph {
} }
}; };
drop(desc);
let res = PipelineResource { let res = PipelineResource {
pipeline, pipeline,
bg_layout_name_lookup: Default::default(), bg_layout_name_lookup: Default::default(),
}; };
self.pipelines.insert(pass.desc.id, res);
let mut pipeline = pass.pipeline.borrow_mut();
*pipeline = Some(res);
} }
} }
} }
@ -282,10 +267,16 @@ impl RenderGraph {
#[instrument(skip(self, world))] #[instrument(skip(self, world))]
pub fn prepare(&mut self, world: &mut World) { pub fn prepare(&mut self, world: &mut World) {
// prepare all passes // prepare all passes
let mut context = RenderGraphContext::new(&self.device, &self.queue, None); let mut buffer_writes = VecDeque::<GraphBufferWrite>::new();
for (_, pass) in &mut self.passes { // reserve some buffer writes. not all nodes write so half the amount of them is probably
// fine.
buffer_writes.reserve(self.passes.len() / 2);
for (label, pass) in &mut self.passes {
let mut context = RenderGraphContext::new(&self.device, &self.queue, None, label.clone());
let mut inner = pass.inner.borrow_mut(); let mut inner = pass.inner.borrow_mut();
inner.prepare(world, &mut context); inner.prepare(world, &mut context);
buffer_writes.append(&mut context.buffer_writes);
} }
{ {
@ -293,10 +284,10 @@ impl RenderGraph {
let s = debug_span!("queue_buffer_writes"); let s = debug_span!("queue_buffer_writes");
let _e = s.enter(); let _e = s.enter();
while let Some(bufwr) = context.buffer_writes.pop_front() { while let Some(bufwr) = buffer_writes.pop_front() {
let slot = self let slot = self
.slots .slots
.get(&self.slot_id_rc(&bufwr.target_slot).unwrap()) .get(&bufwr.target_slot)
.expect(&format!( .expect(&format!(
"Failed to find slot '{:?}' for buffer write", "Failed to find slot '{:?}' for buffer write",
bufwr.target_slot bufwr.target_slot
@ -313,23 +304,22 @@ impl RenderGraph {
#[instrument(skip(self))] #[instrument(skip(self))]
pub fn render(&mut self) { pub fn render(&mut self) {
let mut sorted: VecDeque<u64> = petgraph::algo::toposort(&self.execution_graph, None) let mut sorted: VecDeque<RenderGraphLabelValue> = petgraph::algo::toposort(&self.execution_graph, None)
.expect("RenderGraph had cycled!") .expect("RenderGraph had cycled!")
.iter() .iter()
.map(|i| self.execution_graph[i.clone()]) .map(|i| self.execution_graph[i.clone()].clone())
.collect(); .collect();
let path_names = sorted //debug!("Render graph execution order: {:?}", sorted);
.iter()
.map(|i| self.pass(*i).unwrap().label.clone())
.collect_vec();
trace!("Render graph execution order: {:?}", path_names);
let mut encoders = Vec::with_capacity(self.passes.len() / 2); let mut encoders = Vec::with_capacity(self.passes.len() / 2);
while let Some(pass_id) = sorted.pop_front() { while let Some(pass_label) = sorted.pop_front() {
let pass = self.passes.get(&pass_id).unwrap(); let pass = self.passes.get(&pass_label).unwrap();
let pass_inn = pass.inner.clone(); let pass_inn = pass.inner.clone();
let pass_desc = pass.desc.clone(); let pass_desc = pass.desc.clone();
let label = format!("{:?} Encoder", pass_desc.label); let pass_desc = (*pass_desc).borrow();
let label = format!("{:?} Encoder", pass_label.0);
// encoders are not needed for presenter nodes. // encoders are not needed for presenter nodes.
let encoder = if pass_desc.ty.should_have_pipeline() { let encoder = if pass_desc.ty.should_have_pipeline() {
@ -346,7 +336,7 @@ impl RenderGraph {
// clone of the Rc's is required to appease the borrow checker // clone of the Rc's is required to appease the borrow checker
let device = self.device.clone(); let device = self.device.clone();
let queue = self.queue.clone(); let queue = self.queue.clone();
let mut context = RenderGraphContext::new(&device, &queue, encoder); let mut context = RenderGraphContext::new(&device, &queue, encoder, pass_label.clone());
// all encoders need to be submitted before a presenter node is executed. // all encoders need to be submitted before a presenter node is executed.
if pass_desc.ty == NodeType::Presenter { if pass_desc.ty == NodeType::Presenter {
@ -354,9 +344,9 @@ impl RenderGraph {
self.queue.submit(encoders.drain(..)); self.queue.submit(encoders.drain(..));
} }
trace!("Executing {:?}", pass_desc.label); trace!("Executing {:?}", pass_label.0);
let mut inner = pass_inn.borrow_mut(); let mut inner = pass_inn.borrow_mut();
inner.execute(self, &*pass_desc, &mut context); inner.execute(self, &pass_desc, &mut context);
if let Some(encoder) = context.encoder { if let Some(encoder) = context.encoder {
encoders.push(encoder.finish()); encoders.push(encoder.finish());
@ -374,49 +364,52 @@ impl RenderGraph {
} }
} }
pub fn slot_value(&self, id: u64) -> Option<&SlotValue> { pub fn slot_value<L: Into<RenderGraphLabelValue>>(&self, label: L) -> Option<&SlotValue> {
self.slots.get(&id).map(|s| &s.value) self.slots.get(&label.into()).map(|s| &s.value)
} }
pub fn slot_value_mut(&mut self, id: u64) -> Option<&mut SlotValue> { pub fn slot_value_mut<L: Into<RenderGraphLabelValue>>(&mut self, label: L) -> Option<&mut SlotValue> {
self.slots.get_mut(&id).map(|s| &mut s.value) self.slots.get_mut(&label.into()).map(|s| &mut s.value)
} }
pub fn pass(&self, id: u64) -> Option<&NodeDesc> { pub fn node_desc<L: Into<RenderGraphLabelValue>>(&self, label: L) -> Option<Ref<NodeDesc>> {
self.passes.get(&id).map(|s| &*s.desc) self.passes.get(&label.into()).map(|s| (*s.desc).borrow())
} }
#[inline(always)] #[inline(always)]
pub fn pipeline(&self, id: u64) -> &Pipeline { pub fn pipeline<L: Into<RenderGraphLabelValue>>(&self, label: L) -> Option<Ref<Pipeline>> {
&self.pipelines.get(&id).unwrap().pipeline self.passes.get(&label.into())
.and_then(|p| {
let v = p.pipeline.borrow();
match &*v {
Some(_) => Some(Ref::map(v, |p| &p.as_ref().unwrap().pipeline)),
None => None,
}
})
} }
#[inline(always)] #[inline(always)]
pub fn try_bind_group(&self, id: u64) -> Option<&Rc<wgpu::BindGroup>> { pub fn try_bind_group<L: Into<RenderGraphLabelValue>>(&self, label: L) -> Option<&Rc<wgpu::BindGroup>> {
self.bind_groups.get(&id).map(|e| &e.bg) self.bind_groups.get(&label.into()).map(|e| &e.bg)
} }
#[inline(always)] #[inline(always)]
pub fn bind_group(&self, id: u64) -> &Rc<wgpu::BindGroup> { pub fn bind_group<L: Into<RenderGraphLabelValue>>(&self, label: L) -> &Rc<wgpu::BindGroup> {
self.try_bind_group(id).expect("Unknown id for bind group") self.try_bind_group(label).expect("Unknown id for bind group")
} }
#[inline(always)] #[inline(always)]
pub fn try_bind_group_layout(&self, id: u64) -> Option<&Rc<wgpu::BindGroupLayout>> { pub fn try_bind_group_layout<L: Into<RenderGraphLabelValue>>(&self, label: L) -> Option<&Rc<wgpu::BindGroupLayout>> {
self.bind_groups.get(&id).and_then(|e| e.layout.as_ref()) self.bind_groups.get(&label.into()).and_then(|e| e.layout.as_ref())
} }
#[inline(always)] #[inline(always)]
pub fn bind_group_layout(&self, id: u64) -> &Rc<wgpu::BindGroupLayout> { pub fn bind_group_layout<L: Into<RenderGraphLabelValue>>(&self, label: L) -> &Rc<wgpu::BindGroupLayout> {
self.try_bind_group_layout(id) self.try_bind_group_layout(label)
.expect("Unknown id for bind group layout") .expect("Unknown id for bind group layout")
} }
#[inline(always)]
pub fn bind_group_id(&self, label: &dyn RenderGraphLabel) -> Option<u64> {
self.bind_group_names.get(&label.rc_clone().into()).copied()
}
pub fn add_edge(&mut self, from: impl RenderGraphLabel, to: impl RenderGraphLabel) pub fn add_edge(&mut self, from: impl RenderGraphLabel, to: impl RenderGraphLabel)
{ {
let from = RenderGraphLabelValue::from(from); let from = RenderGraphLabelValue::from(from);
@ -425,13 +418,13 @@ impl RenderGraph {
let from_idx = self let from_idx = self
.passes .passes
.iter() .iter()
.find(|p| p.1.desc.label == from) .find(|p| *p.0 == from)
.map(|p| p.1.graph_index) .map(|p| p.1.graph_index)
.expect("Failed to find from pass"); .expect("Failed to find from pass");
let to_idx = self let to_idx = self
.passes .passes
.iter() .iter()
.find(|p| p.1.desc.label == to) .find(|p| *p.0 == to)
.map(|p| p.1.graph_index) .map(|p| p.1.graph_index)
.expect("Failed to find to pass"); .expect("Failed to find to pass");
@ -471,9 +464,8 @@ impl RenderGraph {
) { ) {
for (label, index) in bind_groups { for (label, index) in bind_groups {
let bg = self let bg = self
.bind_group_id(*label) .bind_group(label.rc_clone());
.map(|bgi| self.bind_group(bgi)) //.expect(&format!("Could not find bind group '{:?}'", label));
.expect(&format!("Could not find bind group '{:?}'", label));
pass.set_bind_group(*index, bg, &[]); pass.set_bind_group(*index, bg, &[]);
} }
@ -490,22 +482,28 @@ pub(crate) struct GraphBufferWrite {
#[allow(dead_code)] #[allow(dead_code)]
pub struct RenderGraphContext<'a> { pub struct RenderGraphContext<'a> {
/// Becomes None when the encoder is submitted /// The [`wgpu::CommandEncoder`] used to encode GPU operations.
pub(crate) encoder: Option<wgpu::CommandEncoder>, ///
pub(crate) device: &'a wgpu::Device, /// This is `None` during the `prepare` stage.
pub(crate) queue: &'a wgpu::Queue, pub encoder: Option<wgpu::CommandEncoder>,
/// The gpu device that is being used.
pub device: &'a wgpu::Device,
pub queue: &'a wgpu::Queue,
pub(crate) buffer_writes: VecDeque<GraphBufferWrite>, pub(crate) buffer_writes: VecDeque<GraphBufferWrite>,
renderpass_desc: Vec<wgpu::RenderPassDescriptor<'a, 'a>>, renderpass_desc: Vec<wgpu::RenderPassDescriptor<'a, 'a>>,
/// The label of this Node.
pub label: RenderGraphLabelValue,
} }
impl<'a> RenderGraphContext<'a> { impl<'a> RenderGraphContext<'a> {
pub(crate) fn new(device: &'a wgpu::Device, queue: &'a wgpu::Queue, encoder: Option<wgpu::CommandEncoder>) -> Self { pub(crate) fn new(device: &'a wgpu::Device, queue: &'a wgpu::Queue, encoder: Option<wgpu::CommandEncoder>, label: RenderGraphLabelValue) -> Self {
Self { Self {
encoder, encoder,
device, device,
queue, queue,
buffer_writes: Default::default(), buffer_writes: Default::default(),
renderpass_desc: vec![], renderpass_desc: vec![],
label,
} }
} }

View File

@ -1,4 +1,4 @@
use std::{cell::{Ref, RefCell, RefMut}, collections::HashMap, num::NonZeroU32, rc::Rc}; use std::{cell::{Ref, RefCell, RefMut}, num::NonZeroU32, rc::Rc};
use bind_match::bind_match; use bind_match::bind_match;
use lyra_ecs::World; use lyra_ecs::World;
@ -97,8 +97,6 @@ pub struct NodeSlot {
pub ty: SlotType, pub ty: SlotType,
/// The way this slot uses the value. Defines if this slot is an output or input. /// The way this slot uses the value. Defines if this slot is an output or input.
pub attribute: SlotAttribute, pub attribute: SlotAttribute,
// What if these are just completely removed and labels are used everywhere instead?
pub id: u64,
/// The identifying label of this slot. /// The identifying label of this slot.
pub label: RenderGraphLabelValue, pub label: RenderGraphLabelValue,
/// The value of the slot. /// The value of the slot.
@ -163,15 +161,12 @@ impl RenderGraphPipelineInfo {
/// Descriptor of a Node in a [`RenderGraph`]. /// Descriptor of a Node in a [`RenderGraph`].
pub struct NodeDesc { pub struct NodeDesc {
pub id: u64,
/// The label of the Node, used to identify the Node.
pub label: RenderGraphLabelValue,
/// The [`NodeType`] of the node. /// The [`NodeType`] of the node.
pub ty: NodeType, pub ty: NodeType,
/// The slots that the Node uses. /// The slots that the Node uses.
/// This defines the resources that the node uses and creates in the graph. /// This defines the resources that the node uses and creates in the graph.
pub slots: Vec<NodeSlot>, pub slots: Vec<NodeSlot>,
slot_label_lookup: HashMap<RenderGraphLabelValue, u64>, //slot_label_lookup: HashMap<RenderGraphLabelValue, u64>,
/// An optional pipeline descriptor for the Node. /// An optional pipeline descriptor for the Node.
/// This is `None` if the Node type is not a node that requires a pipeline /// This is `None` if the Node type is not a node that requires a pipeline
/// (see [`NodeType::should_have_pipeline`]). /// (see [`NodeType::should_have_pipeline`]).
@ -187,19 +182,14 @@ pub struct NodeDesc {
impl NodeDesc { impl NodeDesc {
/// Create a new node descriptor. /// Create a new node descriptor.
pub fn new<L: RenderGraphLabel>( pub fn new(
id: u64,
label: L,
pass_type: NodeType, pass_type: NodeType,
pipeline_desc: Option<PipelineDescriptor>, pipeline_desc: Option<PipelineDescriptor>,
bind_groups: Vec<(&dyn RenderGraphLabel, Rc<wgpu::BindGroup>, Option<Rc<wgpu::BindGroupLayout>>)>, bind_groups: Vec<(&dyn RenderGraphLabel, Rc<wgpu::BindGroup>, Option<Rc<wgpu::BindGroupLayout>>)>,
) -> Self { ) -> Self {
Self { Self {
id,
label: label.into(),
ty: pass_type, ty: pass_type,
slots: vec![], slots: vec![],
slot_label_lookup: HashMap::default(),
pipeline_desc, pipeline_desc,
bind_groups: bind_groups bind_groups: bind_groups
.into_iter() .into_iter()
@ -218,7 +208,6 @@ impl NodeDesc {
"input slots should not have values" "input slots should not have values"
); );
self.slot_label_lookup.insert(slot.label.clone().into(), slot.id);
self.slots.push(slot); self.slots.push(slot);
} }
@ -230,7 +219,6 @@ impl NodeDesc {
#[inline(always)] #[inline(always)]
pub fn add_buffer_slot<L: RenderGraphLabel>( pub fn add_buffer_slot<L: RenderGraphLabel>(
&mut self, &mut self,
id: u64,
label: L, label: L,
attribute: SlotAttribute, attribute: SlotAttribute,
value: Option<SlotValue>, value: Option<SlotValue>,
@ -241,7 +229,6 @@ impl NodeDesc {
); );
let slot = NodeSlot { let slot = NodeSlot {
id,
label: label.into(), label: label.into(),
ty: SlotType::Buffer, ty: SlotType::Buffer,
attribute, attribute,
@ -258,7 +245,6 @@ impl NodeDesc {
#[inline(always)] #[inline(always)]
pub fn add_texture_slot<L: RenderGraphLabel>( pub fn add_texture_slot<L: RenderGraphLabel>(
&mut self, &mut self,
id: u64,
label: L, label: L,
attribute: SlotAttribute, attribute: SlotAttribute,
value: Option<SlotValue>, value: Option<SlotValue>,
@ -269,7 +255,6 @@ impl NodeDesc {
); );
let slot = NodeSlot { let slot = NodeSlot {
id,
label: label.into(), label: label.into(),
ty: SlotType::Texture, ty: SlotType::Texture,
attribute, attribute,
@ -286,7 +271,6 @@ impl NodeDesc {
#[inline(always)] #[inline(always)]
pub fn add_texture_view_slot<L: RenderGraphLabel>( pub fn add_texture_view_slot<L: RenderGraphLabel>(
&mut self, &mut self,
id: u64,
label: L, label: L,
attribute: SlotAttribute, attribute: SlotAttribute,
value: Option<SlotValue>, value: Option<SlotValue>,
@ -297,7 +281,6 @@ impl NodeDesc {
); );
let slot = NodeSlot { let slot = NodeSlot {
id,
label: label.into(), label: label.into(),
ty: SlotType::TextureView, ty: SlotType::TextureView,
attribute, attribute,
@ -314,7 +297,6 @@ impl NodeDesc {
#[inline(always)] #[inline(always)]
pub fn add_sampler_slot<L: RenderGraphLabel>( pub fn add_sampler_slot<L: RenderGraphLabel>(
&mut self, &mut self,
id: u64,
label: L, label: L,
attribute: SlotAttribute, attribute: SlotAttribute,
value: Option<SlotValue>, value: Option<SlotValue>,
@ -325,7 +307,6 @@ impl NodeDesc {
); );
let slot = NodeSlot { let slot = NodeSlot {
id,
label: label.into(), label: label.into(),
ty: SlotType::Sampler, ty: SlotType::Sampler,
attribute, attribute,

View File

@ -40,8 +40,6 @@ pub struct BasePass {
/// This should be Some when the pass is first created then after its added to /// This should be Some when the pass is first created then after its added to
/// the render graph it will be None and stay None. /// the render graph it will be None and stay None.
temp_render_target: Option<RenderTarget>, temp_render_target: Option<RenderTarget>,
main_rt_id: u64,
window_tv_id: u64,
screen_size: glam::UVec2, screen_size: glam::UVec2,
} }
@ -102,8 +100,6 @@ impl Node for BasePass {
let depth_texture_view = Rc::new(depth_texture.view); let depth_texture_view = Rc::new(depth_texture.view);
let mut desc = NodeDesc::new( let mut desc = NodeDesc::new(
graph.next_id(),
BasePassLabel,
NodeType::Node, NodeType::Node,
None, None,
vec![ vec![
@ -119,37 +115,30 @@ impl Node for BasePass {
], ],
); );
self.main_rt_id = graph.next_id();
desc.add_slot(NodeSlot { desc.add_slot(NodeSlot {
ty: SlotType::RenderTarget, ty: SlotType::RenderTarget,
attribute: SlotAttribute::Output, attribute: SlotAttribute::Output,
id: self.main_rt_id,
label: BasePassSlots::MainRenderTarget.into(), label: BasePassSlots::MainRenderTarget.into(),
value: Some(SlotValue::RenderTarget(Rc::new(RefCell::new( value: Some(SlotValue::RenderTarget(Rc::new(RefCell::new(
render_target, render_target,
)))), )))),
}); });
self.window_tv_id = graph.next_id();
desc.add_texture_view_slot( desc.add_texture_view_slot(
self.window_tv_id,
BasePassSlots::WindowTextureView, BasePassSlots::WindowTextureView,
SlotAttribute::Output, SlotAttribute::Output,
Some(SlotValue::Lazy), Some(SlotValue::Lazy),
); );
desc.add_texture_view_slot( desc.add_texture_view_slot(
graph.next_id(),
BasePassSlots::DepthTextureView, BasePassSlots::DepthTextureView,
SlotAttribute::Output, SlotAttribute::Output,
Some(SlotValue::TextureView(depth_texture_view)), Some(SlotValue::TextureView(depth_texture_view)),
); );
desc.add_buffer_slot( desc.add_buffer_slot(
graph.next_id(),
BasePassSlots::ScreenSize, BasePassSlots::ScreenSize,
SlotAttribute::Output, SlotAttribute::Output,
Some(SlotValue::Buffer(Rc::new(screen_size_buf))), Some(SlotValue::Buffer(Rc::new(screen_size_buf))),
); );
desc.add_buffer_slot( desc.add_buffer_slot(
graph.next_id(),
BasePassSlots::Camera, BasePassSlots::Camera,
SlotAttribute::Output, SlotAttribute::Output,
Some(SlotValue::Buffer(Rc::new(camera_buf))), Some(SlotValue::Buffer(Rc::new(camera_buf))),
@ -177,7 +166,7 @@ impl Node for BasePass {
context: &mut crate::render::graph::RenderGraphContext, context: &mut crate::render::graph::RenderGraphContext,
) { ) {
let tv_slot = graph let tv_slot = graph
.slot_value_mut(self.main_rt_id) .slot_value_mut(BasePassSlots::MainRenderTarget)
.expect("somehow the main render target slot is missing"); .expect("somehow the main render target slot is missing");
let mut rt = tv_slot.as_render_target_mut().unwrap(); let mut rt = tv_slot.as_render_target_mut().unwrap();
debug_assert!( debug_assert!(
@ -203,7 +192,7 @@ impl Node for BasePass {
// store the surface texture to the slot // store the surface texture to the slot
let tv_slot = graph let tv_slot = graph
.slot_value_mut(self.window_tv_id) .slot_value_mut(BasePassSlots::WindowTextureView)
.expect("somehow the window texture view slot is missing"); .expect("somehow the window texture view slot is missing");
*tv_slot = SlotValue::TextureView(Rc::new(view)); *tv_slot = SlotValue::TextureView(Rc::new(view));
} }

View File

@ -40,8 +40,6 @@ impl Node for LightBasePass {
let light_buffers = self.light_buffers.as_ref().unwrap(); let light_buffers = self.light_buffers.as_ref().unwrap();
let mut desc = NodeDesc::new( let mut desc = NodeDesc::new(
graph.next_id(),
LightBasePassLabel,
NodeType::Node, NodeType::Node,
None, None,
vec![( vec![(
@ -52,7 +50,6 @@ impl Node for LightBasePass {
); );
desc.add_buffer_slot( desc.add_buffer_slot(
graph.next_id(),
LightBasePassSlots::Lights, LightBasePassSlots::Lights,
SlotAttribute::Output, SlotAttribute::Output,
Some(SlotValue::Buffer(light_buffers.buffer.clone())), Some(SlotValue::Buffer(light_buffers.buffer.clone())),

View File

@ -49,8 +49,7 @@ impl Node for LightCullComputePass {
// get the size of the work group for the grid // get the size of the work group for the grid
let main_rt = graph let main_rt = graph
.slot_id(&BasePassSlots::MainRenderTarget) .slot_value(BasePassSlots::MainRenderTarget)
.and_then(|s| graph.slot_value(s))
.and_then(|s| s.as_render_target()) .and_then(|s| s.as_render_target())
.expect("missing main render target"); .expect("missing main render target");
self.workgroup_size = self.workgroup_size =
@ -169,16 +168,13 @@ impl Node for LightCullComputePass {
})); }));
drop(main_rt); drop(main_rt);
let pass_id = graph.next_id();
let depth_tex_bgl = graph.bind_group_layout(graph.bind_group_id(&BasePassSlots::DepthTexture).unwrap()); let depth_tex_bgl = graph.bind_group_layout(BasePassSlots::DepthTexture);
let camera_bgl = graph.bind_group_layout(graph.bind_group_id(&BasePassSlots::Camera).unwrap()); let camera_bgl = graph.bind_group_layout(BasePassSlots::Camera);
let lights_bgl = graph.bind_group_layout(graph.bind_group_id(&LightBasePassSlots::Lights).unwrap()); let lights_bgl = graph.bind_group_layout(LightBasePassSlots::Lights);
let screen_size_bgl = graph.bind_group_layout(graph.bind_group_id(&BasePassSlots::ScreenSize).unwrap()); let screen_size_bgl = graph.bind_group_layout(BasePassSlots::ScreenSize);
let mut desc = NodeDesc::new( let mut desc = NodeDesc::new(
pass_id,
LightCullComputePassLabel,
NodeType::Compute, NodeType::Compute,
Some(PipelineDescriptor::Compute(ComputePipelineDescriptor { Some(PipelineDescriptor::Compute(ComputePipelineDescriptor {
label: Some("light_cull_pipeline".into()), label: Some("light_cull_pipeline".into()),
@ -201,20 +197,17 @@ impl Node for LightCullComputePass {
); );
desc.add_texture_view_slot( desc.add_texture_view_slot(
graph.next_id(),
BasePassSlots::WindowTextureView, BasePassSlots::WindowTextureView,
SlotAttribute::Input, SlotAttribute::Input,
None, None,
); );
desc.add_buffer_slot( desc.add_buffer_slot(
graph.next_id(),
BasePassSlots::ScreenSize, BasePassSlots::ScreenSize,
SlotAttribute::Input, SlotAttribute::Input,
None, None,
); );
desc.add_buffer_slot(graph.next_id(), BasePassSlots::Camera, SlotAttribute::Input, None); desc.add_buffer_slot(BasePassSlots::Camera, SlotAttribute::Input, None);
desc.add_buffer_slot( desc.add_buffer_slot(
graph.next_id(),
LightCullComputePassSlots::IndexCounterBuffer, LightCullComputePassSlots::IndexCounterBuffer,
SlotAttribute::Output, SlotAttribute::Output,
Some(SlotValue::Buffer(Rc::new(light_index_counter_buffer))), Some(SlotValue::Buffer(Rc::new(light_index_counter_buffer))),
@ -228,27 +221,20 @@ impl Node for LightCullComputePass {
fn execute( fn execute(
&mut self, &mut self,
graph: &mut crate::render::graph::RenderGraph, graph: &mut crate::render::graph::RenderGraph,
desc: &crate::render::graph::NodeDesc, _: &crate::render::graph::NodeDesc,
context: &mut RenderGraphContext, context: &mut RenderGraphContext,
) { ) {
let label = context.label.clone();
let pipeline = graph.pipeline(label)
.expect("Failed to find Pipeline for LightCullComputePass");
let pipeline = pipeline.as_compute();
let mut pass = context.begin_compute_pass(&wgpu::ComputePassDescriptor { let mut pass = context.begin_compute_pass(&wgpu::ComputePassDescriptor {
label: Some("light_cull_pass"), label: Some("light_cull_pass"),
}); });
let pipeline = graph.pipeline(desc.id); pass.set_pipeline(pipeline);
pass.set_pipeline(pipeline.as_compute());
/* let depth_tex_bg = graph.bind_group(graph.bind_group_id("depth_texture").unwrap());
let camera_bg = graph.bind_group(graph.bind_group_id("camera").unwrap());
let lights_bg = graph.bind_group(graph.bind_group_id("light_buffers").unwrap());
let grid_bg = graph.bind_group(graph.bind_group_id("light_indices_grid").unwrap());
let screen_size_bg = graph.bind_group(graph.bind_group_id("screen_size").unwrap());
pass.set_bind_group(0, depth_tex_bg, &[]);
pass.set_bind_group(1, camera_bg, &[]);
pass.set_bind_group(2, lights_bg, &[]);
pass.set_bind_group(3, grid_bg, &[]);
pass.set_bind_group(4, screen_size_bg, &[]); */
graph.set_bind_groups( graph.set_bind_groups(
&mut pass, &mut pass,

View File

@ -232,21 +232,16 @@ impl Node for MeshPass {
self.default_texture = Some(RenderTexture::from_bytes(&device, &graph.queue, texture_bind_group_layout.clone(), bytes, "default_texture").unwrap()); self.default_texture = Some(RenderTexture::from_bytes(&device, &graph.queue, texture_bind_group_layout.clone(), bytes, "default_texture").unwrap());
// get surface config format // get surface config format
let main_rt = graph.slot_id(&BasePassSlots::MainRenderTarget) let main_rt = graph.slot_value(BasePassSlots::MainRenderTarget)
.and_then(|s| graph.slot_value(s))
.and_then(|s| s.as_render_target()) .and_then(|s| s.as_render_target())
.expect("missing main render target"); .expect("missing main render target");
let surface_config_format = main_rt.surface_config.format; let surface_config_format = main_rt.surface_config.format;
drop(main_rt); drop(main_rt);
// get the id here to make borrow checker happy let camera_bgl = graph.bind_group_layout(BasePassSlots::Camera);
let pass_id = graph.next_id(); let lights_bgl = graph.bind_group_layout(LightBasePassSlots::Lights);
let camera_bgl = graph.bind_group_layout(graph.bind_group_id(&BasePassSlots::Camera).unwrap());
let lights_bgl = graph.bind_group_layout(graph.bind_group_id(&LightBasePassSlots::Lights).unwrap());
let light_grid_bgl = graph let light_grid_bgl = graph
.bind_group_layout(graph.bind_group_id(&LightCullComputePassSlots::LightIndicesGridGroup) .bind_group_layout(LightCullComputePassSlots::LightIndicesGridGroup);
.expect("Missing light grid bind group"));
let shader = Rc::new(Shader { let shader = Rc::new(Shader {
label: Some("base_shader".into()), label: Some("base_shader".into()),
@ -254,8 +249,6 @@ impl Node for MeshPass {
}); });
let desc = NodeDesc::new( let desc = NodeDesc::new(
pass_id,
MeshesPassLabel,
NodeType::Render, NodeType::Render,
Some(PipelineDescriptor::Render(RenderPipelineDescriptor { Some(PipelineDescriptor::Render(RenderPipelineDescriptor {
label: Some("meshes".into()), label: Some("meshes".into()),
@ -445,38 +438,37 @@ impl Node for MeshPass {
fn execute( fn execute(
&mut self, &mut self,
graph: &mut crate::render::graph::RenderGraph, graph: &mut crate::render::graph::RenderGraph,
desc: &crate::render::graph::NodeDesc, _: &crate::render::graph::NodeDesc,
context: &mut crate::render::graph::RenderGraphContext, context: &mut crate::render::graph::RenderGraphContext,
) { ) {
let encoder = context.encoder.as_mut().unwrap(); let encoder = context.encoder.as_mut().unwrap();
let view = graph let view = graph
.slot_value(graph.slot_id(&BasePassSlots::WindowTextureView).unwrap()) .slot_value(BasePassSlots::WindowTextureView)
.unwrap() .unwrap()
.as_texture_view() .as_texture_view()
.expect("BasePassSlots::WindowTextureView was not a TextureView slot"); .expect("BasePassSlots::WindowTextureView was not a TextureView slot");
let depth_view = graph let depth_view = graph
.slot_value(graph.slot_id(&BasePassSlots::DepthTextureView).unwrap()) .slot_value(BasePassSlots::DepthTextureView)
.unwrap() .unwrap()
.as_texture_view() .as_texture_view()
.expect("BasePassSlots::DepthTextureView was not a TextureView slot"); .expect("BasePassSlots::DepthTextureView was not a TextureView slot");
let camera_bg = graph let camera_bg = graph
.bind_group(graph.bind_group_id(&BasePassSlots::Camera) .bind_group(BasePassSlots::Camera);
.expect("Missing camera bind group"));
let lights_bg = graph let lights_bg = graph
.bind_group(graph.bind_group_id(&LightBasePassSlots::Lights) .bind_group(LightBasePassSlots::Lights);
.expect("Missing lights bind group"));
let light_grid_bg = graph let light_grid_bg = graph
.bind_group(graph.bind_group_id(&LightCullComputePassSlots::LightIndicesGridGroup) .bind_group(LightCullComputePassSlots::LightIndicesGridGroup);
.expect("Missing light grid bind group"));
let material_bg = graph let material_bg = graph
.bind_group(graph.bind_group_id(&MeshesPassSlots::Material) .bind_group(MeshesPassSlots::Material);
.expect("Missing material bind group"));
let pipeline = graph.pipeline(context.label.clone())
.expect("Failed to find pipeline for MeshPass");
let mut pass = encoder.begin_render_pass(&wgpu::RenderPassDescriptor { let mut pass = encoder.begin_render_pass(&wgpu::RenderPassDescriptor {
label: Some("Render Pass"), label: Some("Render Pass"),
@ -504,7 +496,6 @@ impl Node for MeshPass {
}), }),
}); });
let pipeline = graph.pipeline(desc.id);
pass.set_pipeline(&pipeline.as_render()); pass.set_pipeline(&pipeline.as_render());
//let material_buffer_bg = self.material_buffer.as_ref().unwrap().bindgroup(); //let material_buffer_bg = self.material_buffer.as_ref().unwrap().bindgroup();

View File

@ -18,8 +18,7 @@ impl PresentPassLabel {
/// screen size buffer, camera buffer, /// screen size buffer, camera buffer,
pub struct PresentPass { pub struct PresentPass {
/// Label of this pass /// Label of this pass
label: PresentPassLabel, pub label: PresentPassLabel,
//render_target_slot: Rc<dyn RenderGraphLabel>,
} }
impl PresentPass { impl PresentPass {
@ -32,10 +31,8 @@ impl PresentPass {
} }
impl Node for PresentPass { impl Node for PresentPass {
fn desc(&mut self, graph: &mut crate::render::graph::RenderGraph) -> crate::render::graph::NodeDesc { fn desc(&mut self, _graph: &mut crate::render::graph::RenderGraph) -> crate::render::graph::NodeDesc {
let mut desc = NodeDesc::new( let mut desc = NodeDesc::new(
graph.next_id(),
self.label.clone(),
NodeType::Presenter, NodeType::Presenter,
None, None,
vec![], vec![],
@ -45,7 +42,6 @@ impl Node for PresentPass {
NodeSlot { NodeSlot {
ty: SlotType::RenderTarget, ty: SlotType::RenderTarget,
attribute: SlotAttribute::Input, attribute: SlotAttribute::Input,
id: graph.next_id(),
label: self.label.0.clone(), label: self.label.0.clone(),
value: None, value: None,
} }
@ -59,9 +55,9 @@ impl Node for PresentPass {
} }
fn execute(&mut self, graph: &mut crate::render::graph::RenderGraph, _desc: &crate::render::graph::NodeDesc, _context: &mut crate::render::graph::RenderGraphContext) { fn execute(&mut self, graph: &mut crate::render::graph::RenderGraph, _desc: &crate::render::graph::NodeDesc, _context: &mut crate::render::graph::RenderGraphContext) {
let id = graph.slot_id_rc(&self.label.0) let mut slot = graph.slot_value_mut(self.label.0.clone())
.expect(&format!("render target slot '{:?}' for PresentPass is missing", self.label.0)); .expect(&format!("render target slot '{:?}' for PresentPass is missing", self.label.0))
let mut slot = graph.slot_value_mut(id).unwrap().as_render_target_mut().unwrap(); .as_render_target_mut().unwrap();
let surf_tex = slot.current_texture.take().unwrap(); let surf_tex = slot.current_texture.take().unwrap();
surf_tex.present(); surf_tex.present();
} }

View File

@ -125,19 +125,20 @@ impl BasicRenderer {
let mut g = RenderGraph::new(device.clone(), queue.clone()); let mut g = RenderGraph::new(device.clone(), queue.clone());
debug!("Adding base pass"); debug!("Adding base pass");
g.add_pass(BasePass::new(surface, config)); g.add_pass(BasePassLabel, BasePass::new(surface, config));
debug!("Adding light base pass"); debug!("Adding light base pass");
g.add_pass(LightBasePass::new()); g.add_pass(LightBasePassLabel, LightBasePass::new());
debug!("Adding light cull compute pass"); debug!("Adding light cull compute pass");
g.add_pass(LightCullComputePass::new(size)); g.add_pass(LightCullComputePassLabel, LightCullComputePass::new(size));
//debug!("Adding triangle pass"); //debug!("Adding triangle pass");
//g.add_pass(TrianglePass::new()); //g.add_pass(TrianglePass::new());
debug!("Adding mesh pass"); debug!("Adding mesh pass");
g.add_pass(MeshPass::new()); g.add_pass(MeshesPassLabel, MeshPass::new());
debug!("Adding present pass"); debug!("Adding present pass");
g.add_pass(PresentPass::new(BasePassSlots::MainRenderTarget)); let p = PresentPass::new(BasePassSlots::MainRenderTarget);
g.add_pass(p.label.clone(), p);
g.add_edge(BasePassLabel, LightBasePassLabel); g.add_edge(BasePassLabel, LightBasePassLabel);
g.add_edge(LightBasePassLabel, LightCullComputePassLabel); g.add_edge(LightBasePassLabel, LightCullComputePassLabel);
@ -188,7 +189,7 @@ impl Renderer for BasicRenderer {
self.size = new_size; self.size = new_size;
// update surface config and the surface // update surface config and the surface
let mut rt = self.graph.slot_value_mut(self.graph.slot_id(&BasePassSlots::MainRenderTarget).unwrap()) let mut rt = self.graph.slot_value_mut(BasePassSlots::MainRenderTarget)
.unwrap().as_render_target_mut().unwrap(); .unwrap().as_render_target_mut().unwrap();
rt.surface_config.width = new_size.width; rt.surface_config.width = new_size.width;
rt.surface_config.height = new_size.height; rt.surface_config.height = new_size.height;