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
  • !206

Merged
Created Jun 18, 2021 by Oliver Kirsebom@kirsebomOwner

Gammatone

  • Overview 4
  • Commits 85
  • Pipelines 1
  • Changes 18

This merge request adds the new module gammatone.py which contains the following two classes:

  • GammatoneFilterBank
  • AuralFeatures

This module introduces three new dependencies,

  • Dave/MERIDIAN's aural-features package
  • Scipy>=1.6
  • Python>=3.7

However, at present I don't think we are interested in enforcing these dependencies for all ketos user, especially not the requirement Python>3.7. So, instead I have implemented the following checks and printing of warning messages:

        if 'aural' not in sys.modules:
            try:
                import aural.meridian as au
            except ImportError:
                print('aural-features package not found.')
                print('aural-features is required by the AuralFeatures class.')
                print('install with `pip install aural-features`.')
                print('note that aural-features requires Scipy>=1.6 and Python>=3.7')
                sys.exit(1)   

and,

    if Version(scipy.__version__) < Version("1.6.0"):
        print('The `compute_filter_coeffs` method in the `gammatone` module requires Scipy>=1.6.0')
        print(f'The present environment only has Scipy=={Version(scipy.__version__)}')
        print('Note that Scipy>=1.6.0 requires Python>=3.7.0')
        exit(1)

Note that this merge request also,

  • changes our Python requirement from <3.8 to <=3.8
  • updates our CI script to pull a newly created docker image, which has Scipy 1.6 and Python 3.7
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: nicole