-
Notifications
You must be signed in to change notification settings - Fork 19
Description
The people that maintain the fooof package have renamed it to specparam and are in the process of releasing a new version.
If we don't need the functionality, we should not have a reason to change, BUT, there is a probem: they forgot a random print in their code which is super-annoying when fooof feature is activated since it prints to the console
for bwl in self.peak_width_limits:
print(bwl)So it might be worth to update just to get rid of that.
On the brighter side, thanks to this I realized that the FOOOF model is initialized once per batch, which is probably not necessary, just initialize once and call fit once per batch.
Reference to make the change to the new version: fooof-tools/fooof#205
EDIT: Ok so I realized that the print is not there anymore, maybe because I re-built my environment and it's possible that it was myself who put the print there months ago doing some research into the FOOOF feature. I feel like an idiot.