DAMO-DI-ML / KDD2023-DCdetector

153 stars 16 forks source link

Issue about the instance generation process #15

Open YunhaoZhang-Mars opened 9 months ago

YunhaoZhang-Mars commented 9 months ago

屏幕截图 2023-09-17 175121 Thanks for providing the source code of the great work. I have a question about instance generation: it seems that the parameter 'step' is never used. So the instances on all datasets were generated by a sliding window with step 1. Am I right?

tianzhou2011 commented 9 months ago

That's correct. Unfortunately, we lack sufficient computational resources to perform a grid search for the step size. However, it seems that using a step size of 1 already provides us with a reasonable result.

yyysjz1997 commented 9 months ago

屏幕截图 2023-09-17 175121 Thanks for providing the source code of the great work. I have a question about instance generation: it seems that the parameter 'step' is never used. So the instances on all datasets were generated by a sliding window with step 1. Am I right?

Thanks. In the default configuration, the step size is set to 1 for all datasets except the SMD dataset. The Step parameter is adjusted primarily to enhance the preprocessing's generalization capabilities. If your computing resources are limited, you have the option to fine-tune the step size to expedite model computations, but it may come at the cost of a reduction in model performance.