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

Imputation task #19

Open
chenxiaodanhit opened this issue Aug 28, 2023 · 1 comment
Open

Imputation task #19

chenxiaodanhit opened this issue Aug 28, 2023 · 1 comment

Comments

@chenxiaodanhit
Copy link

Thank you for your valuable contributions! I have some confusion regarding the imputation task. While the code provided showcases the prediction task, it appears that the loss calculation in decoder.py involves generating hist_encoded, pred_encoded, hist_true_x, and pred_true_x using a mask. This seems to imply that the lengths of missing values in a batch are assumed to be constant. However, if the number of missing values in the historical data varies, could you kindly provide suggestions on how to adjust the code to accommodate this scenario? Thank you for your patiance!

@marcotet
Copy link
Collaborator

Thanks for your interest in TACTiS. Sadly, its current architecture doesn't allow for variable length predictions in the decoder.

I don't think reasonable tweaks could allow you to get around this due to the decoder doing a reshuffling of the predicted variables. If I remember correctly, using a variable reshuffling for each batch element had a significant performance impact, and doing so would be the start of having a variable-length prediction window (for example by adding dummy "no need for forecast" variables at the end of the window).

Anyhow, if I was to make a suggestion, it would be to reorder the batching process such that each batch has a constant number of element in the prediction window. The model should be able to handle variable-length prediction windows, as long as each length was in its own batches.

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

2 participants