Error on cyclic import from package hidden in a function

Aperta
#2,418 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
python
Ambito
compilers

Direzione di ricerca

Start by running the reproducer linked in the issue with the shown mypy command, then inspect how package/a.py, package/b.py, and package/init.py are analyzed for cyclic imports and deferred function imports. Compare the package import with the direct package.b import; done means the valid package form no longer produces the reported attribute and undefined-name errors.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug false-positive priority-1-normal topic-import-cycles

I think that mypy incorrectly reports an error. Full steps to reproduce are at https://github.com/lopuhin/mypy-lang-issue-1, but basically, the issue is this: say we have a package package:

package
├── __init__.py
├── a.py
├── b.py
└── test.py

package/a.py has a class that has an import hidden in a function:

class A(object):
    def do(self):
        from package import do_b      # this gives an error
        # from package.b import do_b  # this works
        return do_b(self)

package/b.py defines that do_b function and has an import from package.a in order to have a valid type declaration:

from package.a import A

def do_b(a):
    # type: (A) -> str
    return 'foo'

package/__init__.py exports do_b from b:

from .b import do_b

And package/test.py is just to check that there is no error on the python level:

from package.a import A

a = A()
print(a.do())

Now, the command mypy --silent-imports --check-untyped-defs package gives the following error:

package/a.py: note: In function "do":
package/a.py:3: error: Module 'package' has no attribute 'do_b'
package/a.py:5: error: Name 'do_b' is not defined

But if we import from package.b directly in package/a.py, the error goes away.

Lingua principale
Python
Stelle
20.6k
Fork
3.3k
Merge medio
1g 18h
PR unite (30g)
54

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di python/mypy

Tutte le issue di python/mypy

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.