search

vladmandic / sd-extension-system-info

System and platform info and standardized benchmarking extension for SD.Next and WebUI
https://vladmandic.github.io/sd-extension-system-info/pages/benchmark.html
MIT License
280 stars 51 forks source link

console: UserWarning: You have unused kwarg parameters in XXX #10

Closed davehornik closed 1 year ago

davehornik commented 1 year ago

Can we get a fix for this console spam ?

D:\Stable WEBUI\stable-diffusion-webui\venv\lib\site-packages\gradio\deprecation.py:43: UserWarning: You have unused kwarg parameters in Row, please remove them: {'equal_height': True}
  warnings. Warn(
D:\Stable WEBUI\stable-diffusion-webui\venv\lib\site-packages\gradio\deprecation.py:43: UserWarning: You have unused kwarg parameters in JSON, please remove them: {'lines': 56}
  warnings. Warn(
D:\Stable WEBUI\stable-diffusion-webui\venv\lib\site-packages\gradio\deprecation.py:43: UserWarning: You have unused kwarg parameters in JSON, please remove them: {'lines': 0}
  warnings. Warn(
D:\Stable WEBUI\stable-diffusion-webui\venv\lib\site-packages\gradio\deprecation.py:43: UserWarning: You have unused kwarg parameters in JSON, please remove them: {'lines': 41}
  warnings. Warn(
D:\Stable WEBUI\stable-diffusion-webui\venv\lib\site-packages\gradio\deprecation.py:43: UserWarning: You have unused kwarg parameters in JSON, please remove them: {'lines': 28}
  warnings. Warn(
D:\Stable WEBUI\stable-diffusion-webui\venv\lib\site-packages\gradio\deprecation.py:43: UserWarning: You have unused kwarg parameters in Row, please remove them: {'scale': 0.5}
  warnings. Warn(
vladmandic commented 1 year ago

a) why do you think those messages come from this extension? none of those are actually mentioned anywhere in the codebase. b) there is a good reason why warnings are disabled by default in webui and you should only enable them when you excplicitly require them.

davehornik commented 1 year ago

These warns are not showing while using clean webui without extensions. After enabling just yours they showed up. Thats why I am assuming they are coming from your ext.

vladmandic commented 1 year ago

its actually webui that is sending params to any additional ui components that it attempts to register during startup. so you'd get similar warnings for most of scripts/extensions with any user interface if they are not using those params.

but like i said, warnings are disabled by default and for a good reason - gradio is full of false-positive warnings and normal operations are nearly-impossible with warnings enabled. only reason to enable warnings is if you're doing code development and debugging something specific.