fix(task): model can sometimes be None

[SBNovaScript]

In getter.py, the model param can be an invalid model as a string. In that instance, if Model.from_pretrained() fails silently and returns None, an uncaught error is thrown in this file when we attempt to eval. We can catch this by checking if model can run eval().

I chose to check for the callability of eval rather than a None check to potentially catch any future failures that aren't None but don't contain eval.

[SBNovaScript]

CC @hbredin , please let me know if you'd like any further information. Thanks!

[hbredin]

Thanks. PR should be on develop branch. It says that there are conflicts. Can you please check?

[SBNovaScript]

Thanks. PR should be on develop branch. It says that there are conflicts. Can you please check?

Just rebased, should be all set. Please let me know!

[SBNovaScript]

CC @hbredin Please let me know if you'd like me to change anything, thank you!

[Copilot]

Pull Request Overview

This PR fixes a potential runtime error in the get_model function by adding validation to ensure the loaded model has a callable eval method before attempting to use it. The fix addresses cases where Model.from_pretrained() might fail silently and return None or an invalid object.

Key changes:

• Added validation to check if the model has a callable eval method before invoking it
• Added comprehensive test coverage for the new validation logic

Reviewed Changes

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

File	Description
src/pyannote/audio/pipelines/utils/getter.py	Added validation check for callable eval method with descriptive error message
tests/utils/test_getter.py	Added comprehensive test suite covering edge cases and normal functionality

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

There's a spelling error in the error message. 'Recieved' should be 'Received'.

Suggested change

	f"Recieved: {model}"
	f"Received: {model}"

[Copilot]

There's a spelling error in the test assertion. 'Recieved' should be 'Received' to match the corrected error message.

Suggested change

	assert f"Recieved: {model}" in str(excinfo.value)
	assert f"Received: {model}" in str(excinfo.value)

[Copilot]

There's a spelling error in the test assertion. 'Recieved' should be 'Received' to match the corrected error message.

Suggested change

	assert f"Recieved: {model}" in str(excinfo.value)
	assert f"Received: {model}" in str(excinfo.value)