Sonoff-Tasmota icon indicating copy to clipboard operation
Sonoff-Tasmota copied to clipboard

'ShowSource' was not declared in this scope

Open AndyWo83 opened this issue 5 years ago • 2 comments

Hello,

i would like use the Universal 4 with my Wemos D1 Mini to read my powermeters. But in Arduino i have a error message. How can I fix the error? ShowSource001

AndyWo83 avatar Jun 07 '19 16:06 AndyWo83

Hi Christoph,

Thanks for the PR. I agree that this would be a useful cleanup transformation, but one remark after a brief review: since this gets called as part of the ModelWrapper.cleanup() which essentially gets called after every single transformation, and since the ONNX export behavior we observe from e.g. PyTorch has changed over the course of versions, we should try to add some extra checks for future-proofing here. Could you provide a unit test that exercises this transformation? Might be easiest to take one of the existing ONNX testcases and artificially manipulate the input list as part of the testcase such that the initialized input appears first.

maltanar avatar Feb 23 '24 13:02 maltanar

Could you provide a unit test that exercises this transformation?

Has been added now, testing this transformation on the Sum operator and some input/initializer combinations.

iksnagreb avatar Apr 25 '24 14:04 iksnagreb

LGTM, ready to merge. Thanks @iksnagreb !

maltanar avatar Sep 12 '24 07:09 maltanar