Obscure imports #1

New Issue

c.jansen · 2021-06-25T12:30:19Z

c.jansen commented

2021-06-25 12:30:19 +00:00

lasp / init.py

from .laspcommon import *
from .laspavstream import *
from .wrappers import *
from .laspatomic import *
from .laspimptube import *
from .laspmeasurement import *
from .laspoctavefilter import *
from .laspslm import *
from .lasprecord import *
from .laspsiggen import *
from .laspweighcal import *
from .tools import *

Als je zoekt uit welke submodule een functie afkomstig is omdat je hem wilt aanpassen, is dat niet te doen. Suggesties:

Expliciete imports, dus from .submodule import function1, function2
of: bij elke from .submodule import * commentaar zetten wat er wordt geïmporteerd
Aan het begin van elk script zetten welke functies erin zitten d.m.v.
__all__ = ['function1', 'function2']

**lasp / init.py** ``` from .laspcommon import * from .laspavstream import * from .wrappers import * from .laspatomic import * from .laspimptube import * from .laspmeasurement import * from .laspoctavefilter import * from .laspslm import * from .lasprecord import * from .laspsiggen import * from .laspweighcal import * from .tools import * ``` Als je zoekt uit welke submodule een functie afkomstig is omdat je hem wilt aanpassen, is dat niet te doen. Suggesties: * Expliciete imports, dus `from .submodule import function1, function2` * of: bij elke `from .submodule import *` commentaar zetten wat er wordt geïmporteerd * Aan het begin van elk script zetten welke functies erin zitten d.m.v. `__all__ = ['function1', 'function2']`

anne commented

2021-10-04 13:57:49 +00:00

Klopt, zal even kijken welke functies allemaal nog geen all hebben. Als je een specifieke functie zoekt, kan je ook op de commandline typen:

$ grep -R <text_te_zoeken_text>

Maakt zoekwerk een stuk makkelijker.

Klopt, zal even kijken welke functies allemaal nog geen __all__ hebben. Als je een specifieke functie zoekt, kan je ook op de commandline typen: `$ grep -R <text_te_zoeken_text> ` Maakt zoekwerk een stuk makkelijker.

anne commented

2021-10-04 13:58:31 +00:00

Welke ontbreken er specifiek?

anne self-assigned this 2021-10-04 13:58:42 +00:00

c.jansen commented

2021-10-19 08:21:38 +00:00

Dat weet ik niet meer.

anne closed this issue

2022-10-20 10:36:18 +00:00

Sign in to join this conversation.