ER Classifiers switched to SparseNetworkLBP

CogComp / saul

Saul : Declarative Learning-Based Programming

Other

64 stars 18 forks source link

Closed bhargav closed 8 years ago

bhargav commented 8 years ago

kordjamshidi commented 8 years ago

Thanks, if no one has comments, I am going to merge this.

danyaljj commented 8 years ago

Could you update the results in the readme with this change?

kordjamshidi commented 8 years ago

good point, also do the results change significantly?

bhargav commented 8 years ago

Also fixed #294 issue with populating test instances for Constrained Classifiers.

bhargav commented 8 years ago

The JointTraining results are a bit lower than before. Other experiments have similar results as before.

kordjamshidi commented 8 years ago

Sorry, I should have merged this first, now you need to merge with upstream.

bhargav commented 8 years ago

Closing in lieu of #340 from the public repository.