You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Add the customized_loss_func feature currently available in SAITS to other imputation models.
2. Motivation
Currently saits_loss_func is used in class _Informer (line 96 of pypots/imputation/informer/core.py), indicating that loss function is MAE and cannot be customized with a model parameter (the original paper used MSE). I adjusted locally for my task for using MSE but it would be nice if customized_loss_func can also be added for informer. Besides, it seems some other models are using saits_loss_func in core.py as well (but I didn't check one by one).
3. Your contribution
I can help with updating core.py / model.py but not sure if rolling out the feature for other models are also desired.
The text was updated successfully, but these errors were encountered:
Thank you so much for your attention to PyPOTS! You can follow me on GitHub to receive the latest news of PyPOTS. If you find PyPOTS helpful to your work, please star⭐️ this repository. Your star is your recognition, which can help more people notice PyPOTS and grow PyPOTS community. It matters and is definitely a kind of contribution to the community.
I have received your message and will respond ASAP. Thank you for your patience! 😃
1. Feature description
Add the
customized_loss_func
feature currently available in SAITS to other imputation models.2. Motivation
Currently
saits_loss_func
is used inclass _Informer
(line 96
ofpypots/imputation/informer/core.py
), indicating that loss function is MAE and cannot be customized with a model parameter (the original paper used MSE). I adjusted locally for my task for using MSE but it would be nice ifcustomized_loss_func
can also be added for informer. Besides, it seems some other models are usingsaits_loss_func
incore.py
as well (but I didn't check one by one).3. Your contribution
I can help with updating
core.py
/model.py
but not sure if rolling out the feature for other models are also desired.The text was updated successfully, but these errors were encountered: