fix: pyflakes style

[jorgepiloto]

This is related with #296. It enables the PyFlakes (F) rule. Most of the changes include:

• Remove unused variables
• Remove duplicated tests
• Usage of __all__ in __init__ files

[codecov]

Codecov Report

Attention: Patch coverage is 60.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 87.37%. Comparing base (404befa) to head (b3f4b59).
Report is 149 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1953      +/-   ##
==========================================
- Coverage   88.44%   87.37%   -1.08%     
==========================================
  Files          90       90              
  Lines       10276    10266      -10     
==========================================
- Hits         9089     8970     -119     
- Misses       1187     1296     +109     

[jorgepiloto]

Checking the decrease in coverage. Some tests were duplicated and exactly the same.

[jorgepiloto]

This test slightly differs from another named exactly the same within this file. To be reviewed...

[jorgepiloto]

Progress on this for the next days:

• Restore unused properties
• Verify their value in Pytest

This solves the issues for coverage and codacy.

Example:

mesh = Object.mesh
assert np.assert_allclose(mesh, mesh_expected, atol, rtol)

[moe-ad]

I do not see how we can boost coverage on the erring lines. Most are modified f-strings or exception statements. If the lines are really being covered before now, there is no reason why they shouldn't be covered after the changes, at least I can't think of any.

[moe-ad]

@jorgepiloto , just a reminder that I think we can merge this?

[PProfizi]

Hi @jorgepiloto, hi @moe-ad is this still active? Should it be marked as ready-for-review?

[PProfizi]

Hi @moe-ad @jorgepiloto I'll close the PR as stale. You can reopen on start a new one based on what was proposed here.