Automatically calculate obs_dim based on foresight, unique_obs_dim, ...

[reinecfi]

Description

Currently obs_dim, foresight, unique_obs_dim are fixed values for a given strategy. As they are directly interdependent, obs_dim is now calculated based on the other ones. Adjustments should now only need less changes per strategy.

Also correcting some wrong values in the doc strings and removing the reward scaling in RenewableEnergyLearningSingleBidStrategy (as already done in EnergyLearningStrategy).

Checklist

• Documentation updated (docstrings, READMEs, user guides, inline comments, doc folder updates etc.)
• New unit/integration tests added (if applicable)
• Changes noted in release notes (if any)
• Consent to release this PR's code under the GNU Affero General Public License v3.0

[codecov]

Codecov Report

❌ Patch coverage is 75.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.30%. Comparing base (8f7bd22) to head (5210f1c).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
assume/strategies/learning_strategies.py	57.14%	3 Missing ⚠️
assume/reinforcement_learning/algorithms/matd3.py	85.71%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #708      +/-   ##
==========================================
- Coverage   81.31%   81.30%   -0.01%     
==========================================
  Files          54       54              
  Lines        8032     8035       +3     
==========================================
+ Hits         6531     6533       +2     
- Misses       1501     1502       +1     

Flag	Coverage Δ
pytest	81.30% <75.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[maurerle]

Looks good, but I would like to also wait for feedback from @kim-mskw

[kim-mskw]

Thanks for the nice work @reinecfi. Please adjust the learning notebooks as well; they won't be functional if I am correct, and even if they should reflect the current implementation. I added release notes, noting that any other rationale for the obs_dim needs to be reconfigured in the learning strategy. Please add this explanation to the learning notebook as well.

[reinecfi]

The notebooks should be up to date now.
@kim-mskw Thanks for the updated documentation, I think the additions might be quite helpful.