Replace general exceptions with a specific ones

[suhaibmujahid]

More details here: https://pylint.readthedocs.io/en/latest/user_guide/messages/warning/broad-exception-raised.html

Example of a general exception that should be replaced:

bugbug/bugbug/model.py

Line 192 in 69972a1

raise Exception(f"Unexpected feature type for: {full_feature_name}")

The goal is to replace all general exceptions, not only in the model.py file.

[prernadabi23]

@suhaibmujahid Please assign this issue to me, I would like to work on it.

[Emeka-Onwuepe]

@suhaibmujahid I am a bit curious, the get_feature_names method in the Model Class will always return an empty list. Is this the ideal implementation?

[suhaibmujahid]

@suhaibmujahid Please assign this issue to me, I would like to work on it.

You do not need to ask. Please start by reading #1092

@suhaibmujahid I am a bit curious, the get_feature_names method in the Model Class will always return an empty list. Is this the ideal implementation?

The get_feature_names() will not be called directly; it is a placeholder that will be overridden. Alternatively, returning an empty list could be replaced by raisingNotImplementedError.

However, this question is off-topic. For simmiler questions, you could open a new issue, use Discussions or chat with us in the bugbug Matrix room.

[Emeka-Onwuepe]

Thanks for the reply and the corrections are duly noted.

[Munaelekwa]

#3343