search

dswah / sgcrfpy

SGCRFpy: Sparse Gaussian Conditional Random Fields in Python
MIT License
29 stars 5 forks source link

remove all Gammas #5

Open dswah opened 8 years ago

dswah commented 8 years ago

they show up in grad_wrt_theta and maybe more places

dswah commented 8 years ago

Is this computation necessary? can we leverage a previous matrix product?

dswah commented 8 years ago

there are lots of implicit Gamma computations in the check descent loop. these should be cached if possible!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

dswah commented 8 years ago

FIX THIS!! the coord descent loop is so wasteful >:0 !!!