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

Double normalization #17

Open
Yiwen233 opened this issue Jun 29, 2024 · 4 comments
Open

Double normalization #17

Yiwen233 opened this issue Jun 29, 2024 · 4 comments

Comments

@Yiwen233
Copy link

Yiwen233 commented Jun 29, 2024

Thanks for your work. I read the source code of Informer with RevIN, and I found that the data is normalized by standard normalization, and then sent to the RevIn layer. So there are actually two normalization steps by default. So is that the way to enhance performance and get the result in your paper?

@PlanckChang
Copy link

I also found this issue. Do you have any new ideas recently?

@khchul
Copy link

khchul commented Sep 11, 2024

Yeah I also think this is a big issue that the authors should answer. Because RevIN is casually used in many recent Time Series papers, the double normalization problem appears very frequently. I'm also wondering if the result of the paper could change if the scale variable in the dataloader is set to false.

@Yiwen233
Copy link
Author

I also found this issue. Do you have any new ideas recently?

Not yet

@SamKnightGit
Copy link

I would also be interested in the Author's weighing in on this. Table 3 in the paper seems to suggest that RevIN replaces standard normalization techniques but if standardization is necessary for the RevIN layer to reproduce the results, this should probably be made explicit. I plan on replicating a subset of the results with NBEATS on the ETTh1 dataset, with and without pre-standardization of the input data, I'll update here with my results.

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

No branches or pull requests

4 participants