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

Questions about Event Discovery Algorithm #12

Closed
kibbon opened this issue Jan 18, 2019 · 1 comment
Closed

Questions about Event Discovery Algorithm #12

kibbon opened this issue Jan 18, 2019 · 1 comment

Comments

@kibbon
Copy link

kibbon commented Jan 18, 2019

Hello, I've read your paper "Deep Sequence Learning with Auxiliary Information for Traffic Prediction" and find that the code of Event Discovery Algorithm may be not in accordance with its describtion in paper.
@ /src/preprocessing/new_anomalty1109.py line 38-43
The code shows that rules to discover event is:
(d(x,y,t) - d(x,y,t-7d))) > 300 && (d(x,y,t) / d(x,y,t-7d))) > 0.2 (The second condition make no sense)
In paper this rule to discover event is :
(d(x,y,t) - d(x,y,t-7d))) > 300 && (d(x,y,t) - d(x,y,t-7d))/d(x,y,t-7d) > 0.2

@bbliao
Copy link
Collaborator

bbliao commented Jan 18, 2019

@kibbon Yes, there is discordance between the paper and code. I am sorry for the carelessness since when my partner showed me the result, everything seems ok, then I did not review the code carefully. The difference is that the rule in the code will discover a little more events. (The "events" discovered by the code and their query counts are more than 300/0.2=1500 may not be "events" in the paper.)

So, If you just want to use the event discovery algorithm for event discovery, I suggest you use the rule in the paper. If you want to use our dataset and compare with our results, I suggest you use the code we provided.

I am sorry for the discordance again! Thanks for your attention!

@bbliao bbliao closed this as completed Mar 21, 2019
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