Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SamFormer multivariate Estimator #3196

Open
wants to merge 14 commits into
base: dev
Choose a base branch
from
Open

SamFormer multivariate Estimator #3196

wants to merge 14 commits into from

Conversation

kashif
Copy link
Contributor

@kashif kashif commented Jun 15, 2024

@kashif
Copy link
Contributor Author

kashif commented Jun 17, 2024

@romilbert this implementation differs, I believe, from yours in that I am using the mu and sigma from the instance normalization as inputs to the model as well. This serves the purpose of RevIN + non-stationary transformer and was first done in the DeepAR paper, which as far as I remember the RevIN or the SAMFromer paper does not mention....

Apart from that the output head is an independent distribution head instead of a point forecasting head...

@romilbert
Copy link

Hi @kashif, thanks for your interest in SAMformer. I appreciate your feedback. Incorporating mu and sigma from RevIN as additional inputs to the model is an interesting idea (similar to the DeepAR paper as you said, which we did reference in SAMformer paper). However, I have tried incorporating them into the transformer in different ways, and it did not improve our current results.

@kashif
Copy link
Contributor Author

kashif commented Jun 17, 2024

@romilbert that is because, I think, you are using globally standardized data while here the data continues to be in the original magnitude, and thus in your setup the use of the summary statistics do not help as they are more or less 0 and 1...

@kashif kashif changed the title SamFormer model SamFormer multivariate Estimator Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants