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

About the script example #48

Closed
HugoGuillen opened this issue Feb 27, 2025 · 3 comments
Closed

About the script example #48

HugoGuillen opened this issue Feb 27, 2025 · 3 comments
Labels

Comments

@HugoGuillen
Copy link

First of all, thank you for all your hard work! This project is very interesting.

I have a comment regarding your script example:

X = StandardScaler().fit_transform(X.to_numpy())
X = X.reshape(num_samples, 48, -1)
X_ori = X  # keep X_ori for validation
X = mcar(X, 0.1)  # randomly hold out 10% observed values as ground truth

Please correct me if I'm wrong, but If you take the hold out AFTER scaling, you already leaked the data distribution to the model training, isn't it?
In that case you, one would need to scale after taking the hold out, and then use these parameters to scale the hold out using those parameters to make a fair evaluation.

Best,
Hugo Guillen.

@WenjieDu
Copy link
Owner

Hi there,

Thank you so much for your attention to SAITS! If you find SAITS is helpful to your work, please star⭐️ this repository. Your star is your recognition, which can let others notice SAITS. It matters and is definitely a kind of contribution.

I have received your message and will respond ASAP. Thank you again for your patience! 😃

Best,
Wenjie

@WenjieDu
Copy link
Owner

Hi @HugoGuillen, yes, you're right. It could result in information leakage to the training model.

Copy link

This issue had no activity for 14 days. It will be closed in 1 week unless there is some new activity. Is this issue already resolved?

@github-actions github-actions bot added the stale label Mar 14, 2025
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants