-
Notifications
You must be signed in to change notification settings - Fork 224
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Factor out backend (eg PyTorch, TensorFlow) validation. #230
Comments
I would like to solve this issue. I am new to open source therefore If I make any mistakes please do guide me. |
@amanjha8100 for sure! The most appropriate place for this function seems to be https://github.com/SeldonIO/alibi-detect/blob/master/alibi_detect/utils/frameworks.py. The function should take alibi-detect/alibi_detect/cd/classifier.py Lines 97 to 102 in 13b2664
BACKENDS=['tensorflow', 'pytorch'] constant in this module.
@arnaudvl we may need additional functionality in the future, e.g. if some methods implement one backend but not the other. |
I will try to solve this issue. Thank you for the guidance. |
This issue will be solved by #537. |
Add
_validate_backend
functionality instead of checking on the fly as in https://github.com/SeldonIO/alibi-detect/blob/master/alibi_detect/cd/classifier.py#L98.The text was updated successfully, but these errors were encountered: