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
First, an attempt to provide docstrings to the repository was disastrous. The documentation makes very little sense in isolation, and there is no top-level documentation that pulls it all together.
Second, the code is rife with poor programming practices, like single-letter variable names, which only exacerbate the first issue.
The remediation of this issue will likely span multiple changes. It's possible that the best outcome is a refactor towards typing (as opposed to the nested dict/tuple kerfuffle that exists today. The best outcome is not a requirement for closing this issue.
The text was updated successfully, but these errors were encountered:
Adapt has two core readability problems:
First, an attempt to provide docstrings to the repository was disastrous. The documentation makes very little sense in isolation, and there is no top-level documentation that pulls it all together.
Second, the code is rife with poor programming practices, like single-letter variable names, which only exacerbate the first issue.
The remediation of this issue will likely span multiple changes. It's possible that the best outcome is a refactor towards typing (as opposed to the nested dict/tuple kerfuffle that exists today. The best outcome is not a requirement for closing this issue.
The text was updated successfully, but these errors were encountered: