search

bschiffthaler / seidr

Toolkit to create crowd networks
GNU General Public License v3.0
2 stars 1 forks source link

Removing `--scale,-s` option from seidr and make scaling default #25

Closed bschiffthaler closed 4 years ago

bschiffthaler commented 4 years ago

I propose that we remove the --scale,-s option from Seidr algorithms where it has been shown that scaling usually improves predictions (All ensemble regression + pearson correlation) and instead introduce a --no-scale option for advanced users.

Since this is an API-breaking change, I'll put it up here for discussion.

IMO, seidr should provide "best-effort" defaults

bschiffthaler commented 4 years ago

As per the internal discussion, the option --scale will be made default where applicable. Setting --scale will have no effect. A new option --no-scale will be added to turn scaling off if desired