Remove error_message_searchspace_fully_observed message in BO

[stijnh]

Currently, BO throws an exception once it has fully explored the search space. This means it cannot be used for small(er) search spaces, as there is no way to catch this exception and still get the results from Kernel Tuner.

This PR replaces these exception by just breaking from the loop.

@fjwillemsen is this change ok? Could it have any adverse effects?

[fjwillemsen]

Yes, this looks good to me.

[fjwillemsen]

Good change, this should be handled by the interface, not individual search strategies, so this is definitely more appropriate. Although if the search space is so small as to be fully observed BO should probably not have been used in the first place.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[stijnh]

Although if the search space is so small as to be fully observed BO should probably not have been used in the first place.

I fully agree! However, when setting time limit, you can't know in advance if that's sufficient to explore the full search space.

I will merge this PR now.