init: Fix F401 Linter Warnings by Replacing Wildcard Imports with Explicit Imports in __init__.py

[arohanajit]

Addresses multiple F401 linter warnings in the __init__.py file by replacing wildcard imports with explicit imports and defining the __all__ variable to specify the public API of the module. Replaced Wildcard Imports: Removed from .data import * and explicitly imported the necessary functions and classes. Also updated .flake8 since no F403 warnings were found

[petrasovaa]

You have several duplications in all...

[petrasovaa]

You need to run isort and the CI fails because there is missing Popen class from core.py

[petrasovaa]

The CI fails:

  File "scripts/t.vect.import", line 113, in main
    tgis.import_stds(
  File "etc/python/grass/temporal/stds_import.py", line 250, in import_stds
    old_state = gs.raise_on_error
AttributeError: module 'grass.script' has no attribute 'raise_on_error'. Did you mean: 'get_raise_on_error'?

because there is missing raise_on_error from core.py. But I would say in this case it should use the getter anyway:

old_state = gs.get_raise_on_error()

So the variable probably should not be exposed.

[arohanajit]

The CI fails:

  File "scripts/t.vect.import", line 113, in main
    tgis.import_stds(
  File "etc/python/grass/temporal/stds_import.py", line 250, in import_stds
    old_state = gs.raise_on_error
AttributeError: module 'grass.script' has no attribute 'raise_on_error'. Did you mean: 'get_raise_on_error'?

because there is missing raise_on_error from core.py. But I would say in this case it should use the getter anyway:

old_state = gs.get_raise_on_error()

So the variable probably should not be exposed.

So in this case CI will probably always fail unless we expose 'raise_on_error()` which we shouldn't. Is there another way to manage this?

[petrasovaa]

The CI fails:

  File "scripts/t.vect.import", line 113, in main
    tgis.import_stds(
  File "etc/python/grass/temporal/stds_import.py", line 250, in import_stds
    old_state = gs.raise_on_error
AttributeError: module 'grass.script' has no attribute 'raise_on_error'. Did you mean: 'get_raise_on_error'?

because there is missing raise_on_error from core.py. But I would say in this case it should use the getter anyway:

old_state = gs.get_raise_on_error()

So the variable probably should not be exposed.

So in this case CI will probably always fail unless we expose 'raise_on_error()` which we shouldn't. Is there another way to manage this?

I was suggesting to do the change in stds_import.py

[echoix]

The .flake8 changes mentioned in the PR description were lost in one of the commits. I tried cleaning it back locally, iterating by using pre-commit run -a flake8 between each change. So the init.py exclusions can be limited to python/grass/temporal.