Loop and test restructuring

[carmocca]

What does this PR do?

Makes the following changes

@@ -4,9 +4,7 @@
 │   │   ├── base.py
 │   │   ├── batch
 │   │   │   ├── __init__.py
-│   │   │   ├── manual.py
 │   │   │   └── training_batch_loop.py
-│   │   ├── closure.py
 │   │   ├── dataloader
 │   │   │   ├── __init__.py
 │   │   │   ├── dataloader_loop.py
@@ -18,8 +16,10 @@
 │   │   │   ├── prediction_epoch_loop.py
 │   │   │   └── training_epoch_loop.py
 │   │   ├── fit_loop.py
-│   │   ├── optimizer
+│   │   ├── optimization
 │   │   │   ├── __init__.py
+│   │   │   ├── closure.py
+│   │   │   ├── manual_loop.py
 │   │   │   └── optimizer_loop.py
 │   │   └── utilities.py
 └── tests
@@ -28,16 +28,15 @@
     │   ├── batch
     │   │   ├── __init__.py
     │   │   └── test_truncated_bptt.py
-    │   ├── test_closure.py
-    │   ├── test_loop_state_dict.py
-    │   └── test_loops.py
-    ├── trainer
-    │   ├── loops
+    │   ├── optimization
     │   │   ├── __init__.py
-    │   │   ├── test_all.py
-    │   │   ├── test_evaluation_loop.py
-    │   │   ├── test_evaluation_loop_flow.py
-    │   │   ├── test_flow_warnings.py
-    │   │   ├── test_training_loop.py
-    │   │   ├── test_training_loop_flow_dict.py
-    │   │   └── test_training_loop_flow_scalar.py
+    │   │   └── test_closure.py
+    │   ├── test_all.py
+    │   ├── test_evaluation_loop.py
+    │   ├── test_evaluation_loop_flow.py
+    │   ├── test_flow_warnings.py
+    │   ├── test_loop_state_dict.py
+    │   ├── test_loops.py
+    │   ├── test_training_loop.py
+    │   ├── test_training_loop_flow_dict.py
+    │   └── test_training_loop_flow_scalar.py

Does your PR introduce any breaking changes? If yes, please list them.

None user facing

Before submitting

• Was this discussed/approved via a GitHub issue? (not for typos and docs)
• Did you read the contributor guideline, Pull Request section?
• Did you make sure your PR does only one thing, instead of bundling different changes together?
• [n/a] Did you make sure to update the documentation with your changes? (if necessary)
• [n/a] Did you write any new necessary tests? (not for typos and docs)
• Did you verify new and existing tests pass locally with your changes?
• Did you list all the breaking changes introduced by this pull request?
• [n/a] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review

• Is this pull request ready for review? (if not, please submit in draft mode)
• Check that all items from Before submitting are resolved
• Make sure the title is self-explanatory and the description concisely explains the PR
• Add labels and milestones (and optionally projects) to the PR so it can be classified

[tchaton]

LGTM !

[codecov]

Codecov Report

Merging #9383 (87f8278) into master (e0f2e04) will increase coverage by 0%.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #9383   +/-   ##
======================================
  Coverage      93%     93%           
======================================
  Files         179     180    +1     
  Lines       14922   14965   +43     
======================================
+ Hits        13861   13904   +43     
  Misses       1061    1061