Skip to content

Add barWidth/barDepth params to Graph3DBars; document alpha#1873

Merged
mscuthbert merged 2 commits intomasterfrom
graph3d-configuration
Apr 20, 2026
Merged

Add barWidth/barDepth params to Graph3DBars; document alpha#1873
mscuthbert merged 2 commits intomasterfrom
graph3d-configuration

Conversation

@mscuthbert
Copy link
Copy Markdown
Member

@mscuthbert mscuthbert commented Apr 20, 2026

  • Adds explicit barWidth and barDepth keyword arguments to Graph3DBars.__init__ so callers can set fixed bar dimensions without using per-point format dicts.
  • Documents the previously undocumented alpha, barWidth, and barDepth parameters in the class docstring

This PR is needed to be able to recreate graphs that look like the old music21 2010 paper's graphs.

AI-assisted

mscuthbert and others added 2 commits April 19, 2026 16:05
Adds explicit `barWidth` and `barDepth` keyword arguments to
`Graph3DBars.__init__` so callers can set fixed bar dimensions
without needing per-point format dicts.  When `None` (default)
the existing automatic formulas are used.  Also documents the
previously undocumented `alpha` parameter and refactors
`renderSubplot` to avoid redundant multi-pass iteration.

AI-assisted.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 93.071% (-0.003%) from 93.074% — graph3d-configuration into master

@mscuthbert mscuthbert merged commit e9a708a into master Apr 20, 2026
7 of 8 checks passed
@mscuthbert mscuthbert deleted the graph3d-configuration branch April 20, 2026 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants