-
-
Notifications
You must be signed in to change notification settings - Fork 11.1k
MAINT: removed unused imports listed in LGTM #19090
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,3 @@ | ||
import sys | ||
|
||
from numpy.distutils.fcompiler import FCompiler | ||
|
||
compilers = ['NVHPCFCompiler'] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,7 +15,6 @@ | |
import textwrap | ||
|
||
# Overwrite certain distutils.ccompiler functions: | ||
import numpy.distutils.ccompiler # noqa: F401 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's possible this import is needed to do some monkey-patching. I don't know for sure. |
||
from numpy.distutils import log | ||
# NT stuff | ||
# 1. Make sure libpython<version>.a exists for gcc. If not, build it. | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -33,7 +33,6 @@ | |
import builtins | ||
|
||
# needed in this module for compatibility | ||
from numpy.lib.histograms import histogram, histogramdd | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. idem., See the comment on the line above. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should these be added to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Might be worthwhile to figure out why exactly they're here in the first place. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah, found it: four years ago they were moved from |
||
|
||
|
||
array_function_dispatch = functools.partial( | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,3 @@ | ||
import sys | ||
import os | ||
import re | ||
import functools | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The fact these are imported here is considered public API for this module