Drop toposort and migrate to graphlib #1820
Labels
Dependencies
Pull requests that update a dependency file
Issue: Feature Request
Python
Pull requests that update Python code
Description
This ticket aims at dropping toposort dependency in favor of built-in graphlib if possible. Following how Kedro team did this - kedro-org/kedro#3728
Context
We were depending on Kedro's environment for toposort dependency and since Kedro dropped this dependency, Viz started failing. This also raises an issue of having transitive dependencies. We can improve the code base to remove such transitive dependencies in future - #1821
Possible Implementation
We use
from toposort import CircularDependencyError, toposort_flatten
in our layers module. We need to find an alternate implementation totoposort_flatten
within graphlib to resolve this issuePossible Alternatives
Checklist
The text was updated successfully, but these errors were encountered: