-
Notifications
You must be signed in to change notification settings - Fork 116
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
Error of LogAnomaly implementation. #14
Comments
Hey @YangLin-George, Have you got an answer to this? |
Hi, @asimokby , I'm not sure what is the "answer" you mean. But according to the discussion with authors of LogAnomaly, they did not open-source their implementation due to several concerns. This project is the only one I can find in GitHub which combines DeepLog, LogAnomaly and LogRobust. As for the inappropriate implementation of LogAnomaly. I was debugging LogAnomaly, and found out that the size of the input tensor of loganomaly model in I'm not sure what the id (1 in the input size) is, but seems like there's one step missing which maps event ids into high-demensional vectors in I'm currently not interested in LogAnomaly after we discussed with the authors. So, if you want more details, you can reach out to them directly. If you are studying log anomaly detection, There's one paper accepted by ICSE2021 that you may be interested. If you have any other question, you can email me directly, my email address is linyang[AT]tju.edu.cn. Best, |
Hi@YangLin-George By the way, why you are not interested in LogAnomaly? I am currently doing research on deep learning-based log anomaly detection and maybe you could leave me an email and we can discuss a little. And I was wondering if you could please tell me which paper you mentioned in ICSE2021. |
Hi @KiteFlyKid , |
After reading the original LogAnomaly paper, I found that this implementation is somewhat incorrect. In the LogAnomaly paper, they indeed use SemanticVector and CountVector as inputs for their LSTM model. However, they original authors did not describe how to use "attention" to connect these two inputs in their original paper. That is a big pity that they did not open-source their code! |
Hi @LeonYang95 Im trying to email you but the email format doesn't seem to be correct. How can I reach you? |
Hi @alexjamesmx , you can try [email protected] |
Hi,
Thank you so much for this amazing project!
I am recently playing with different methods here in this project, however, I do find something odd about the implementation of LogAnomaly. Here in this project, LogAnomaly is actually using log event ids within a window to predict the next event id, just the same as DeepLog, which seems wrong.
According to the paperwork of LogAnomaly, it seems using "semantic vectors" for the prediction (the prediction is an event id or semantic vector, either way is good).
So, as far as I can see, should we change the inputs of LogAnomaly from
Sequentials
andQuantitives
toSemantics
andQuantitives
?Please correct me if I misunderstood something. Thanks again for sharing the project!
Lin, Yang
The text was updated successfully, but these errors were encountered: