search

SeldonIO / alibi-detect

Algorithms for outlier, adversarial and drift detection
https://docs.seldon.io/projects/alibi-detect/en/stable/
Other
2.21k stars 220 forks source link

Error related to 'scale_identity_multiplier' in tensorflow #820

Closed nys3015 closed 1 year ago

nys3015 commented 1 year ago

I found 'TypeError: MultivariateNormalDiag.An error such as init() got an unexpected keyword argument 'scale_identity_multiplier' . These errors are found in the 38th line of (alibi_detect/models/tensorflow/losses.py ).

Referring to the document in 'tensorflow' (https://www.tensorflow.org/probability/api_docs/python/tfp/distributions/MultivariateNormalDiag) , we found that the 'multivariatenormalDiag' in tensorflow no longer uses scale_identity_multiplier.

In this regard, it seems that a quick correction is needed.

ascillitoe commented 1 year ago

Hi @nys3015, thank you for raising this. We will work on a fix.

ascillitoe commented 1 year ago

Note to team:

Looks like scale_identify_multiplier was removed from tensorflow-probability in this commit, and released in v0.20.0.

ascillitoe commented 1 year ago

Hi again @nys3015. I just realised we've actually seen this issue in our tests already (See comment), but only in tensorflow-probability v0.20.0, which we don't officially support yet.

Are you using tensorflow-probability v0.20.0 by any chance? If so, are you able to downgrade to v0.19? As a side-note, could I check how you installed tensorflow please? If you installed it via pip install alibi-detect[tensorflow] I would have thought pip would ensure only compatible versions are installed.