Allow subcoordinates to be drawn from NdOverlay dimensions

[philippjfr]

Resolves #6028

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.53%. Comparing base (346be14) to head (95f2c25).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6209   +/-   ##
=======================================
  Coverage   88.52%   88.53%           
=======================================
  Files         323      323           
  Lines       67906    67932   +26     
=======================================
+ Hits        60116    60142   +26     
  Misses       7790     7790           

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

[philippjfr]

This should be good to merge. I will investigate whether adding support for subcoordinate groups drawn from the NdOverlay dimensions makes sense as a separate PR.

[hoxbro]

Tests pass and look like a relatively simple change.

[droumis]

This doesn't seem to work for subcoordinate_y with the wide df example posted here

df = pd.read_csv('https://datasets.holoviz.org/stocks/v1/stocks.csv', parse_dates=['Date']).set_index('Date')

hv.NdOverlay({col: hv.Curve(df, 'Date', (col, 'Price')).opts(tools=['hover'], subcoordinate_y=True) for col in df.columns}, 'Ticker')

[droumis]

If I zoom out, the lines appear, so looks like a ranging issue.

Screen.Recording.2024-06-10.at.11.18.01.AM.mov

[philippjfr]

Thanks, investigating.

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.