Skip to content

Fix drawdown label checks in PerformanceMetrics class - #7

Open
Yosri-Ben-Halima wants to merge 3 commits into
mainfrom
hotfix-drawdown-duration-formatting
Open

Yosri-Ben-Halima wants to merge 3 commits into
mainfrom
hotfix-drawdown-duration-formatting

Conversation

@Yosri-Ben-Halima

Copy link
Copy Markdown
Collaborator

Update label checks to include "Max Drawdown" and "Average Drawdown" for accurate performance formatting.

@konradbachusz

konradbachusz commented Nov 16, 2025

Copy link
Copy Markdown
Owner

@yosri-bh can we please include specific versions of packages in requirements.txt?

For example, instead of seaborn~=0.13.2 we have seaborn==0.13.2

This helps with the stability of the solution.

Can you also please edit this example usage if needed, because currently it doesn't show the Alpha and Beta graphs properly? Also, I don't think the metrics report is shown as you intended.
https://github.com/konradbachusz/algorithmic-trading-utilities?tab=readme-ov-file#performance-metrics

Yosri Ben Halima added 2 commits November 17, 2025 23:02
@Yosri-Ben-Halima Yosri-Ben-Halima self-assigned this Nov 17, 2025
@Yosri-Ben-Halima Yosri-Ben-Halima added bug Something isn't working enhancement New feature or request labels Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants