polara icon indicating copy to clipboard operation
polara copied to clipboard

Notebook 'Comparing LightFM with HybridSVD.ipynb'

Open rodfloripa opened this issue 2 years ago • 3 comments

I tried to run https://github.com/evfro/polara/blob/master/examples/Comparing%20LightFM%20with%20HybridSVD.ipynb, had to change the variable catch=None to catch=(RuntimeWarning,). It doesn't run if you don't make this change.

rodfloripa avatar Mar 16 '22 18:03 rodfloripa

Commas are missing here: all_scores = { 'SVD (best)': svd.evaluate() 'LightFM (best)': evaluate_lightfm(lfm) f'HybridSVD (rank {hsvd.rank})': hsvd.evaluate() }

Corrected all_scores = { 'SVD (best)': svd.evaluate(), 'LightFM (best)': evaluate_lightfm(lfm), f'HybridSVD (rank {hsvd.rank})': hsvd.evaluate() }

rodfloripa avatar Mar 21 '22 13:03 rodfloripa

Replace : trials_df = (study.trials_dataframe().loc[:, ['value', 'params', 'user_attrs']].rename(columns={'': target_metric})) To: trials_df = study.trials_dataframe().loc[:, ['value', 'params_rank', 'user_attrs_metrics','user_attrs_epochs']].rename(columns={'': target_metric})

rodfloripa avatar Mar 22 '22 16:03 rodfloripa

Hi! Thanks for reporting!

I'll have a look. There's a long-time-delayed update in my internal develop branch, I'll include the necessary modifications in the forthcoming release.

evfro avatar Mar 26 '22 05:03 evfro