search

cdt15 / lingam

Python package for causal discovery based on LiNGAM.
https://sites.google.com/view/sshimizu06/lingam
MIT License
380 stars 59 forks source link

change `issubclass` to `isinstance` #25

Closed xiangyu-sun-789 closed 3 years ago

xiangyu-sun-789 commented 3 years ago

Hi,

Thank you so much for providing this helpful LINGAM package. I was looking at how to use ICA-LINAM with VAR-LINGAM. However, the code throws errors when I do so: ICALiNGAM_model = lingam.ICALiNGAM() VARLiNGAM_model = lingam.VARLiNGAM(lags=1, lingam_model=ICALiNGAM_model) One way to fix it is to change issubclass to isinstance.

Thank you very much.

ikeuchi-screen commented 3 years ago

@xiangyu-sun-789 Thanks for finding the bug and sending the patch. I'll merge it right away.