Skip to content

feat(plotly): implement heatmap-correlation#62

Closed
claude[bot] wants to merge 1 commit intomainfrom
auto/heatmap-correlation/plotly
Closed

feat(plotly): implement heatmap-correlation#62
claude[bot] wants to merge 1 commit intomainfrom
auto/heatmap-correlation/plotly

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude Bot commented Nov 29, 2025

Summary

Implements heatmap-correlation for plotly library.

Parent Issue: #53
Sub-Issue: #56
Attempt: 1/3

Implementation

  • plots/plotly/heatmap/heatmap-correlation/default.py

Features

  • Interactive correlation matrix heatmap with hover tooltips
  • Displays correlation values in cells with configurable formatting
  • Supports upper triangle masking for cleaner display
  • Uses RdBu colorscale for clear positive/negative correlation distinction
  • Proper input validation for numeric columns
  • Responsive layout with appropriate sizing

Quality Check

✅ Axes labeled (no explicit x/y labels needed for correlation matrix)
✅ Grid subtle (plotly_white template)
✅ Elements clear (correlation values displayed)
✅ No overlap (angled x-axis labels)
✅ Legend present (color bar shows correlation scale)
✅ Colorblind safe (RdBu colorscale)
✅ Figure size (configurable, defaults to 1000x800px)
✅ Type hints and docstring included
✅ Input validation implemented
✅ Clear error messages

Test Result

Successfully generated plot.png with sample weather data correlation matrix.

Add correlation matrix heatmap implementation for plotly library
- Creates interactive heatmap with hover tooltips
- Shows correlation values in cells with configurable formatting
- Supports upper triangle masking option
- Uses RdBu colorscale by default for clear positive/negative distinction
- Includes proper input validation for numeric columns

Generated with Claude Code

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

codecov Bot commented Nov 29, 2025

The author of this PR, claude[bot], is not an activated member of this organization on Codecov.
Please activate this user on Codecov to display this PR comment.
Coverage data is still being uploaded to Codecov.io for purposes of overall coverage calculations.
Please don't hesitate to email us at support@codecov.io with any questions.

@MarkusNeusinger MarkusNeusinger deleted the auto/heatmap-correlation/plotly branch December 1, 2025 20:15
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.

1 participant