Add warning when a default value isn't parsed

Author: 0HyperCube

Cargo.lock

@@ -17,12 +17,8 @@ fn text<'i: 'n>(
 	#[unit(" px")]
 	#[default(0.)]
 	character_spacing: f64,
-	#[unit(" px")]
-	#[default(None)]
-	max_width: Option<f64>,
-	#[unit(" px")]
-	#[default(None)]
-	max_height: Option<f64>,
+	#[unit(" px")] max_width: Option<f64>,
+	#[unit(" px")] max_height: Option<f64>,
 	/// Faux italic.
 	#[unit("°")]
 	#[default(0.)]

node-graph/gstd/src/text.rs

@@ -7,6 +7,7 @@ license = "MIT OR Apache-2.0"
 [features]
 
 [dependencies]
+log = { workspace = true }
 
 # Workspace dependencies
 graphene-std = { workspace = true, features = ["gpu"] }

node-graph/preprocessor/Cargo.toml

@@ -1,3 +1,6 @@
+#[macro_use]
+extern crate log;
+
 use graph_craft::document::value::*;
 use graph_craft::document::*;
 use graph_craft::proto::RegistryValueSource;
@@ -150,7 +153,14 @@ pub fn node_inputs(fields: &[registry::FieldMetadata], first_node_io: &NodeIOTyp
 
 			match field.value_source {
 				RegistryValueSource::None => {}
-				RegistryValueSource::Default(data) => return NodeInput::value(TaggedValue::from_primitive_string(data, ty).unwrap_or(TaggedValue::None), exposed),
+				RegistryValueSource::Default(data) => {
+					if let Some(custom_default) = TaggedValue::from_primitive_string(data, ty) {
+						return NodeInput::value(custom_default, exposed);
+					} else {
+						// It is incredibly useful to get a warning when the default type cannot be parsed rather than defaulting to `()`.
+						warn!("Failed to parse default value for type {ty:?} with data {data}");
+					}
+				}
 				RegistryValueSource::Scope(data) => return NodeInput::scope(Cow::Borrowed(data)),
 			};