Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This removes unused imports, both in the current code (
npc_gzip/
,examples/
, andtests/
) and in the old code (original_codebase/
).Either all or nearly all unused imports are removed, depending on precisely how one defines "unused." I avoided removing imports (or making any changes) where imports are unreferenced but in some other sense used, such that removing them without making other code changes would cause breakage or alter behavior.
Info on "unused" imports that are needed and not removed (click to expand if interested)
Although major changes probably shouldn't be made to the original codebase without strong reason, it is clear from context that the unused imports in it are mostly due to other fairly recent changes (some as recent as #30).
I have tested this change, both in the new code by running
pytest
in thepoetry
-managed environment (in Python 3.11, but also CI checks this for 3.9, 3.10, and 3.11), and in the original codebase by installing its dependencies in a Python 3.10 virtual environment and runningpython main_text.py
andpython main_text.py --para
.