Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • ketos ketos
  • Project information
    • Project information
    • Activity
    • Labels
    • Planning hierarchy
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 27
    • Issues 27
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • public_projects
  • ketosketos
  • Merge requests
  • !251

Merged
Created Jun 10, 2022 by Bruno Padovese@bpadoveseOwner

Fixes to spectrogram display and waveform pad

  • Overview 0
  • Commits 9
  • Pipelines 4
  • Changes 4

This merge requests has 3 changes:

  1. Mag and power Spectrogram now properly plot a spectrogram without dB label when decibel=False. I decided against adding this option as a class attribute as these changes would only really affect these two spectrgorams... But it could be worth considering if we think it makes sense.
  2. To compute the stft, sometimes the time-series needs to be padded. There are two modes for this, pad_mode='zero' or 'reclect'. This mode is now properly propagated through the code when creating spectrograms
  3. Better warning describing this pad process
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: fix_spectrogram_decibel_label