Add the annotator to the list of valid keywords

related to bob.bio.face!41 (merged)

I don't understand why this check exists. This function can be used in so many different places for different purposes but this single line does not allow it.

This test is mainly because I want to make sure that keywords are not misspelled. If you do not have this test, the error message will be very cryptic: https://gitlab.idiap.ch/bob/bob.bio.base/blob/317a66e6cf6102c83f743395d71c69e6227fb889/bob/bio/base/utils/resources.py#L178

I have spent some time debugging one such issue, where I simply had misspelled one of the keywords.

I will turn this into a warning then.

changed this line in version 2 of the diff

I can see this being turned into a warning. It might just be annoying when you want to use this function in a different way and obtain a warning each time.

so what do I do?

Honestly, anything is fine for me: removing the test, turning it into a warning, keeping it as it is. Your choice.

added 1 commit

• a4b2e31d - use a warning instead of an error

Compare with previous version

closed