Skip to content

Better date axes#68

Open
phil-brad wants to merge 7 commits into
masterfrom
feature/heatmap-date-ticks
Open

Better date axes#68
phil-brad wants to merge 7 commits into
masterfrom
feature/heatmap-date-ticks

Conversation

@phil-brad
Copy link
Copy Markdown
Member

@phil-brad phil-brad commented May 22, 2026

When all values of an axis are datetimes, then use the matplotlib concise date formatter for a nice date axis. This is a non-breaking change with regards to Sift (i.e. won't break anything without the pending PR in Sift, so can be merged first )

Copilot AI review requested due to automatic review settings May 22, 2026 20:07
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves matplotlib axis tick formatting by detecting when axis labels are datetime-like and switching to Matplotlib’s concise date formatting to produce more readable date axes.

Changes:

  • Added datetime-axis detection for x_labels/y_labels and introduced date-specific tick generation/formatting helpers.
  • Updated tick rotation/alignment handling for the x-axis.
  • Initialized grid line state during widget construction.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pyqttoolkit/views/plot/matplotlib/base.py Outdated
Comment thread pyqttoolkit/views/plot/matplotlib/base.py
Comment thread pyqttoolkit/views/plot/matplotlib/base.py Outdated
Comment thread pyqttoolkit/views/plot/matplotlib/base.py
Comment thread pyqttoolkit/views/plot/matplotlib/base.py Outdated
Comment thread pyqttoolkit/views/plot/matplotlib/base.py Outdated
Comment thread pyqttoolkit/views/plot/matplotlib/base.py
Comment thread pyqttoolkit/views/plot/matplotlib/base.py Outdated
phil-brad and others added 2 commits May 26, 2026 10:56
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@phil-brad phil-brad requested a review from NoePonceH May 26, 2026 11:11
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