smcclennon / SongID

Music identification Telegram bot
https://t.me/SongIDBot
Apache License 2.0
87 stars 20 forks source link

Attempt to fix ACRAPI config broken in last commit #13

Closed smcclennon closed 1 week ago

smcclennon commented 1 week ago

Summary by Sourcery

Bug Fixes:

sourcery-ai[bot] commented 1 week ago

Reviewer's Guide by Sourcery

This PR fixes a configuration issue in the ACRAPI module by correctly accessing configuration values from a dictionary structure instead of using direct variable references.

Updated class diagram for ACRAPI configuration

classDiagram
    class ACRAPI {
        +noisy(filePath)
        +hum(filePath)
    }
    class ACRCloudRecognizer {
        +ACRCloudRecognizer(config)
    }
    ACRAPI --> ACRCloudRecognizer
    note for ACRAPI "Configuration is now accessed via a dictionary key ['noisy'] or ['hum'] instead of direct variable reference."

File-Level Changes

Change Details Files
Fixed configuration access in audio recognition methods
  • Changed direct config variable references to dictionary access
  • Updated noisy() method to use config['noisy'] instead of config_noisy
  • Updated hum() method to use config['hum'] instead of config_hum
app/ACRAPI.py

Tips and commands #### Interacting with Sourcery - **Trigger a new review:** Comment `@sourcery-ai review` on the pull request. - **Continue discussions:** Reply directly to Sourcery's review comments. - **Generate a GitHub issue from a review comment:** Ask Sourcery to create an issue from a review comment by replying to it. - **Generate a pull request title:** Write `@sourcery-ai` anywhere in the pull request title to generate a title at any time. - **Generate a pull request summary:** Write `@sourcery-ai summary` anywhere in the pull request body to generate a PR summary at any time. You can also use this command to specify where the summary should be inserted. #### Customizing Your Experience Access your [dashboard](https://app.sourcery.ai) to: - Enable or disable review features such as the Sourcery-generated pull request summary, the reviewer's guide, and others. - Change the review language. - Add, remove or edit custom review instructions. - Adjust other review settings. #### Getting Help - [Contact our support team](mailto:support@sourcery.ai) for questions or feedback. - Visit our [documentation](https://docs.sourcery.ai) for detailed guides and information. - Keep in touch with the Sourcery team by following us on [X/Twitter](https://x.com/SourceryAI), [LinkedIn](https://www.linkedin.com/company/sourcery-ai/) or [GitHub](https://github.com/sourcery-ai).