Skip to content

Add frequency axis in review tab#946

Merged
Josef-Haupt merged 1 commit into
mainfrom
938-add-frequency-scale-to-spectrograms-in-review-mode
Jul 6, 2026
Merged

Add frequency axis in review tab#946
Josef-Haupt merged 1 commit into
mainfrom
938-add-frequency-scale-to-spectrograms-in-review-mode

Conversation

@Josef-Haupt

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 6, 2026 15:22
@Josef-Haupt Josef-Haupt linked an issue Jul 6, 2026 that may be closed by this pull request
@Josef-Haupt Josef-Haupt merged commit b27c3a6 into main Jul 6, 2026
7 checks passed
@Josef-Haupt Josef-Haupt deleted the 938-add-frequency-scale-to-spectrograms-in-review-mode branch July 6, 2026 15:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds an option to display a frequency (y-axis) scale on generated spectrogram figures and enables it in the GUI review tab, so reviewers can interpret spectrograms with an explicit frequency reference.

Changes:

  • Introduced a show_freq_axis flag to spectrogram generation utilities and propagated it from file-based to audio-based rendering.
  • Updated spectrogram rendering to optionally show a linear frequency axis with kHz tick formatting and labels.
  • Enabled the frequency axis for spectrograms displayed in the review tab.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
birdnet_analyzer/utils.py Adds show_freq_axis support and updates spectrogram plotting to optionally render a frequency y-axis.
birdnet_analyzer/gui/review.py Turns on show_freq_axis for spectrograms shown in the review workflow UI.

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

Comment thread birdnet_analyzer/utils.py
matplotlib.ticker.FuncFormatter(lambda x, _: f"{x / 1000:g}")
)
ax.set_ylabel("Frequency (kHz)")
f.tight_layout(pad=.5)
Comment thread birdnet_analyzer/utils.py
Comment on lines 101 to 105
import matplotlib.pyplot as plt
import matplotlib.ticker
import numpy as np

matplotlib.use("agg")
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.

Add Frequency Scale to Spectrograms in Review Mode

2 participants