Skip to content

Commit d794aed

Browse files
jeremygoochpsychedelicious
authored andcommitted
fix(ui): sets cfg_rescael_multiplier to 0 if there is no default. Also fixes issue with truthiness check causing 0 value to be missed. See #7584
1 parent 9930440 commit d794aed

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

invokeai/frontend/web/src/app/store/middleware/listenerMiddleware/listeners/setDefaultSettings.ts

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import type { AppStartListening } from 'app/store/middleware/listenerMiddleware';
2+
import { isNil } from 'es-toolkit';
23
import { bboxHeightChanged, bboxWidthChanged } from 'features/controlLayers/store/canvasSlice';
34
import { selectIsStaging } from 'features/controlLayers/store/canvasStagingAreaSlice';
45
import {
@@ -86,10 +87,16 @@ export const addSetDefaultSettingsListener = (startAppListening: AppStartListeni
8687
}
8788
}
8889

89-
if (cfg_rescale_multiplier) {
90+
if (!isNil(cfg_rescale_multiplier)) {
9091
if (isParameterCFGRescaleMultiplier(cfg_rescale_multiplier)) {
9192
dispatch(setCfgRescaleMultiplier(cfg_rescale_multiplier));
9293
}
94+
} else {
95+
// Set this to 0 if it doesn't have a default. This value is
96+
// easy to miss in the UI when users are resetting defaults
97+
// and leaving it non-zero could lead to detrimental
98+
// effects.
99+
dispatch(setCfgRescaleMultiplier(0));
93100
}
94101

95102
if (steps) {

0 commit comments

Comments
 (0)