Previously, dependency cycles that were created during node expansion
would cause the application to quit (due to an uncaught exception). Now,
we'll throw a proper error to the UI. We also make an attempt to 'blame'
the most relevant node in the UI.
Additionally, if `VALIDATE_INPUTS` takes an argument named `input_types`,
that variable will be a dictionary of the socket type of all incoming
connections. If that argument exists, normal socket type validation will
not occur. This removes the last hurdle for enabling variant types
entirely from custom nodes, so I've removed that command-line option.
I've added appropriate unit tests for these changes.
* Add inference tests
* Clean up
* Rename test graph file
* Add readme for tests
* Separate server fixture
* test file name change
* Assert images are generated
* Clean up comments
* Add __init__.py so tests can run with command line `pytest`
* Fix command line args for pytest
* Loop all samplers/schedulers in test_inference.py
* Ci quality workflows compare (#1)
* Add image comparison tests
* Comparison tests do not pass with empty metadata
* Ensure tests are run in correct order
* Save image files with test name
* Update tests readme
* Reduce step counts in tests to ~halve runtime
* Ci quality workflows build (#2)
* Add build test github workflow
Jacob Segal
comfyanonymous
enzymezoo-code