Added information to the matrix to be able to quickly determine if a parameter has an input assigned.

This commit is contained in:
Weird Constructor 2021-06-30 06:22:38 +02:00
parent 7201ec315e
commit c94a000115

View file

@ -15,6 +15,8 @@ pub use crate::monitor::MON_SIG_CNT;
use crate::matrix_repr::*;
use crate::dsp::tracker::PatternData;
use std::collections::HashSet;
/// This is a cell/tile of the hexagonal [Matrix].
///
/// The [Matrix] stores it to keep track of the graphical representation
@ -295,6 +297,12 @@ pub struct Matrix {
/// by [Matrix::sync] and [Matrix::check].
edges: Vec<Edge>,
/// Stores the [dsp::ParamId] of the inputs that have an output
/// assigned to them. It's updates when [Matrix::edges] is updated and used
/// by [Matrix::param_input_is_used] to return whether a parameter is
/// controlled by some output port.
assigned_inputs: HashSet<ParamId>,
/// Holds the currently monitored cell.
monitored_cell: Cell,
@ -317,6 +325,7 @@ impl Matrix {
saved_matrix: None,
graph_ordering: NodeGraphOrdering::new(),
edges: Vec::with_capacity(MAX_ALLOCATED_NODES * 2),
assigned_inputs: HashSet::new(),
config,
w,
h,
@ -453,6 +462,7 @@ impl Matrix {
self.graph_ordering.clear();
self.edges.clear();
self.assigned_inputs.clear();
self.saved_matrix = None;
self.config.delete_nodes();
@ -748,6 +758,10 @@ impl Matrix {
Some(&self.matrix[x * self.h + y])
}
pub fn param_input_is_used(&self, p: ParamId) -> bool {
self.assigned_inputs.contains(&p)
}
pub fn get_unused_instance_node_id(&self, id: NodeId) -> NodeId {
self.config.unused_instance_node_id(id)
}
@ -791,6 +805,7 @@ impl Matrix {
fn update_graph_ordering_and_edges(&mut self) {
self.graph_ordering.clear();
self.edges.clear();
self.assigned_inputs.clear();
for x in 0..self.w {
for y in 0..self.h {
@ -848,6 +863,12 @@ impl Matrix {
}
}
}
for edge in self.edges.iter() {
if let Some(pid) = edge.to.param_by_idx(edge.to_input as usize) {
self.assigned_inputs.insert(pid);
}
}
}
/// Compiles a [NodeProg] from the data collected by the previous
@ -1027,6 +1048,43 @@ mod tests {
assert_eq!(prog.prog[2].to_string(), "Op(i=2 out=(2-3) in=(4-6) at=(0-0) cpy=(o1 => i4))");
}
#[test]
fn check_matrix_param_is_used() {
use crate::nodes::new_node_engine;
let (node_conf, _node_exec) = new_node_engine();
let mut matrix = Matrix::new(node_conf, 3, 3);
matrix.place(0, 0,
Cell::empty(NodeId::Sin(0))
.out(None, Some(0), None));
matrix.place(1, 0,
Cell::empty(NodeId::Sin(1))
.input(None, Some(0), None)
.out(None, None, Some(0)));
matrix.place(1, 1,
Cell::empty(NodeId::Sin(2))
.input(Some(0), None, None));
matrix.sync().unwrap();
assert!(
matrix.param_input_is_used(
NodeId::Sin(1).inp_param("freq").unwrap()));
assert!(
!matrix.param_input_is_used(
NodeId::Sin(0).inp_param("freq").unwrap()));
matrix.place(1, 0, Cell::empty(NodeId::Nop));
matrix.sync().unwrap();
assert!(
!matrix.param_input_is_used(
NodeId::Sin(1).inp_param("freq").unwrap()));
assert!(
!matrix.param_input_is_used(
NodeId::Sin(2).inp_param("freq").unwrap()));
}
#[test]
fn check_matrix_filled() {
use crate::nodes::new_node_engine;