Skip to content

Commit 30181f3

Browse files
authored
Merge pull request #461 from ranaroussi/fix-issue-454-chart-naming
Fix chart naming to indicate cumulative returns (fixes #454)
2 parents 488984c + e32984f commit 30181f3

3 files changed

Lines changed: 9 additions & 1 deletion

File tree

CHANGELOG.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,13 @@
11
Changelog
22
===========
33

4+
0.0.70
5+
------
6+
7+
- Fixed chart naming inconsistency: renamed "Daily Active Returns" to "Daily Active Returns (Cumulative Sum)" and "Daily Returns" to "Daily Returns (Cumulative Sum)" to accurately reflect that charts show cumulative values (fixes issue #454)
8+
- Fixed CAGR calculation bug where years were incorrectly calculated using calendar days instead of trading periods, causing drastically reduced CAGR values (fixes issue #458)
9+
- Fixed inconsistent EOY returns for benchmarks by preserving original benchmark data for aggregation while aligning to strategy index for other calculations (fixes issue #457)
10+
411
0.0.69
512
------
613

quantstats/_plotting/wrappers.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -812,6 +812,7 @@ def daily_returns(
812812

813813
# Set plot title based on active returns setting
814814
plot_title = "Daily Active Returns" if active else "Daily Returns"
815+
plot_title += " (Cumulative Sum)"
815816

816817
# Use core plotting function for daily time series
817818
fig = _core.plot_timeseries(

quantstats/version.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
version = "0.0.69"
1+
version = "0.0.70"

0 commit comments

Comments
 (0)