From 02a0eeff1e0881654ec24d36a8e6655c0f59e81d Mon Sep 17 00:00:00 2001
From: Tino Koch <17991193+Tinoooo@users.noreply.github.com>
Date: Mon, 6 Jan 2025 21:36:37 +0100
Subject: [PATCH 1/5] chore: removed prop defaults for scanline effect,
 improved blendMode watcher

---
 src/core/pmndrs/ScanlinePmndrs.vue | 28 +++++++++++++++++++---------
 1 file changed, 19 insertions(+), 9 deletions(-)

diff --git a/src/core/pmndrs/ScanlinePmndrs.vue b/src/core/pmndrs/ScanlinePmndrs.vue
index 5246c4b3..f485d913 100644
--- a/src/core/pmndrs/ScanlinePmndrs.vue
+++ b/src/core/pmndrs/ScanlinePmndrs.vue
@@ -1,4 +1,5 @@
 <script lang="ts" setup>
+import { watch } from 'vue'
 import type { BlendFunction } from 'postprocessing'
 import { ScanlineEffect } from 'postprocessing'
 import { makePropWatchers } from '../../util/prop'
@@ -26,14 +27,7 @@ export interface ScanlinePmndrsProps {
   opacity?: number
 }
 
-const props = withDefaults(
-  defineProps<ScanlinePmndrsProps>(),
-  {
-    density: 1.25,
-    opacity: 1.0,
-    scrollSpeed: 0.0,
-  },
-)
+const props = defineProps<ScanlinePmndrsProps>()
 
 const { pass, effect } = useEffectPmndrs(() => new ScanlineEffect(props), props)
 
@@ -42,11 +36,27 @@ defineExpose({ pass, effect })
 makePropWatchers(
   [
     [() => props.blendFunction, 'blendMode.blendFunction'],
-    [() => props.opacity, 'blendMode.opacity.value'],
     [() => props.density, 'density'],
     [() => props.scrollSpeed, 'scrollSpeed'],
   ],
   effect,
   () => new ScanlineEffect(),
 )
+
+watch(
+  [() => props.opacity],
+  () => {
+    if (props.opacity !== undefined) {
+      effect.value?.blendMode.setOpacity(props.opacity)
+    }
+    else {
+      const plainEffect = new ScanlineEffect()
+      effect.value?.blendMode.setOpacity(plainEffect.blendMode.getOpacity())
+      plainEffect.dispose()
+    }
+  },
+  {
+    immediate: true,
+  },
+)
 </script>

From 275209e70e978b7228f649e10ed7ce2fbc053364 Mon Sep 17 00:00:00 2001
From: Tino Koch <17991193+Tinoooo@users.noreply.github.com>
Date: Mon, 6 Jan 2025 21:50:44 +0100
Subject: [PATCH 2/5] - removed defaults from chromatic aberration effect -
 added workaround

---
 src/core/pmndrs/ChromaticAberration.vue | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/src/core/pmndrs/ChromaticAberration.vue b/src/core/pmndrs/ChromaticAberration.vue
index f424d911..07109a91 100644
--- a/src/core/pmndrs/ChromaticAberration.vue
+++ b/src/core/pmndrs/ChromaticAberration.vue
@@ -1,9 +1,9 @@
 <script lang="ts" setup>
-import type { BlendFunction } from 'postprocessing'
 import { ChromaticAberrationEffect } from 'postprocessing'
-import { Vector2 } from 'three'
 import { makePropWatchers } from '../../util/prop'
 import { useEffectPmndrs } from './composables/useEffectPmndrs'
+import type { Vector2 } from 'three'
+import type { BlendFunction } from 'postprocessing'
 
 export interface ChromaticAberrationPmndrsProps {
   /**
@@ -30,13 +30,21 @@ export interface ChromaticAberrationPmndrsProps {
 const props = withDefaults(
   defineProps<ChromaticAberrationPmndrsProps>(),
   {
-    offset: () => new Vector2(0.01, 0.01),
-    radialModulation: false,
-    modulationOffset: 0.15,
+    radialModulation: undefined,
   },
 )
 
-const { pass, effect } = useEffectPmndrs(() => new ChromaticAberrationEffect(props), props)
+const plainEffect = new ChromaticAberrationEffect()
+
+const { pass, effect } = useEffectPmndrs(() => new ChromaticAberrationEffect({
+  ...props,
+  // Unfortunately, these defaults must be set this way as the type in postprocessing is not correct.
+  // The arguments are optional in the actual constructor, but not in the type.
+  radialModulation: props.radialModulation ?? plainEffect.radialModulation,
+  modulationOffset: props.modulationOffset ?? plainEffect.modulationOffset,
+}), props)
+
+plainEffect.dispose()
 
 defineExpose({ pass, effect })
 

From 23a29882af4823c7eb45d01bf837adabdca90d32 Mon Sep 17 00:00:00 2001
From: Tino Koch <17991193+Tinoooo@users.noreply.github.com>
Date: Mon, 6 Jan 2025 21:53:23 +0100
Subject: [PATCH 3/5] remove default prop values from HueSaturationPmndrs
 component

---
 src/core/pmndrs/HueSaturationPmndrs.vue | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/core/pmndrs/HueSaturationPmndrs.vue b/src/core/pmndrs/HueSaturationPmndrs.vue
index 518dbb47..b1482a77 100644
--- a/src/core/pmndrs/HueSaturationPmndrs.vue
+++ b/src/core/pmndrs/HueSaturationPmndrs.vue
@@ -24,13 +24,7 @@ export interface HueSaturationPmndrsProps {
 
 }
 
-const props = withDefaults(
-  defineProps<HueSaturationPmndrsProps>(),
-  {
-    saturation: 0.0,
-    hue: 0.0,
-  },
-)
+const props = defineProps<HueSaturationPmndrsProps>()
 
 const { pass, effect } = useEffectPmndrs(() => new HueSaturationEffect(props), props)
 

From 6fb13b41f881156e90a350e9f0a0e00fd033f350 Mon Sep 17 00:00:00 2001
From: Tino Koch <17991193+Tinoooo@users.noreply.github.com>
Date: Mon, 6 Jan 2025 21:55:29 +0100
Subject: [PATCH 4/5] chore: removed default prop values for NoisePmndrs
 component

---
 src/core/pmndrs/NoisePmndrs.vue | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/core/pmndrs/NoisePmndrs.vue b/src/core/pmndrs/NoisePmndrs.vue
index 217684a0..3432f9dc 100644
--- a/src/core/pmndrs/NoisePmndrs.vue
+++ b/src/core/pmndrs/NoisePmndrs.vue
@@ -1,6 +1,7 @@
 <script lang="ts">
 import { useLoop } from '@tresjs/core'
-import { BlendFunction, NoiseEffect } from 'postprocessing'
+import type { BlendFunction } from 'postprocessing'
+import { NoiseEffect } from 'postprocessing'
 import { omit } from '../../util/object'
 import { makePropWatchersUsingAllProps } from '../../util/prop'
 import { useEffectPmndrs } from './composables/useEffectPmndrs'
@@ -16,8 +17,7 @@ export interface NoisePmndrsProps {
 
 <script lang="ts" setup>
 const props = withDefaults(defineProps<NoisePmndrsProps>(), {
-  premultiply: false,
-  blendFunction: BlendFunction.SCREEN,
+  premultiply: undefined,
 })
 
 const { pass, effect } = useEffectPmndrs(() => new NoiseEffect(props), props)

From 4a7d624d552401071de5b2f330cc0b137b9efb46 Mon Sep 17 00:00:00 2001
From: Tino Koch <17991193+Tinoooo@users.noreply.github.com>
Date: Mon, 6 Jan 2025 21:57:07 +0100
Subject: [PATCH 5/5] chore: removed default prop values for VignettePmndrs
 component

---
 src/core/pmndrs/VignettePmndrs.vue | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/core/pmndrs/VignettePmndrs.vue b/src/core/pmndrs/VignettePmndrs.vue
index 38129cbd..6e0c1f93 100644
--- a/src/core/pmndrs/VignettePmndrs.vue
+++ b/src/core/pmndrs/VignettePmndrs.vue
@@ -1,5 +1,6 @@
 <script lang="ts">
-import { BlendFunction, VignetteEffect, VignetteTechnique } from 'postprocessing'
+import type { BlendFunction, VignetteTechnique } from 'postprocessing'
+import { VignetteEffect } from 'postprocessing'
 import { omit } from '../../util/object'
 import { makePropWatchersUsingAllProps } from '../../util/prop'
 import { useEffectPmndrs } from './composables/useEffectPmndrs'
@@ -10,18 +11,13 @@ export interface VignettePmndrsProps {
    */
   technique?: VignetteTechnique
   blendFunction?: BlendFunction
-  offset: number
-  darkness: number
+  offset?: number
+  darkness?: number
 }
 </script>
 
 <script lang="ts" setup>
-const props = withDefaults(defineProps<VignettePmndrsProps>(), {
-  technique: VignetteTechnique.DEFAULT,
-  blendFunction: BlendFunction.NORMAL,
-  offset: 0.5,
-  darkness: 0.5,
-})
+const props = defineProps<VignettePmndrsProps>()
 
 const { pass, effect } = useEffectPmndrs(() => new VignetteEffect(props), props)
 defineExpose({ pass, effect })