You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm currently going through the code to understand what's going on.
For reference, the current dependency graph in the project looks as follows (assume #84 gets merged; created using pydeps):
To be completely honest, I am very much confused by what does what. So I propose the following: refactor the code.
Step 1
As a first step, I'd like to remove ssort._utils and move its functions to different locations:
Right now there are the following functions defined in _utils.py:
I'm currently going through the code to understand what's going on.
For reference, the current dependency graph in the project looks as follows (assume #84 gets merged; created using
pydeps
):To be completely honest, I am very much confused by what does what. So I propose the following: refactor the code.
Step 1
As a first step, I'd like to remove
ssort._utils
and move its functions to different locations:Right now there are the following functions defined in
_utils.py
:Each of these functions is only used in very few places:
I propse to move these functions to the following files:
The resulting graph looks as follows:
#91 implements these changes.
What do you think?
The text was updated successfully, but these errors were encountered: