Rework log analysis#728
Conversation
Signed-off-by: QazCetelic <dev@malplena.net>
|
@TheKodeToad any more changes needed before this can be added? |
|
@maskersss, sorry for pinging you, but could you perhaps take a look? |
TheKodeToad
left a comment
There was a problem hiding this comment.
I'm not much of a Rust developer (got to admit, seeing this code after writing some C++ is making me tempted), but I'm not seeing any obvious major issues
We'll probably want to audit the code of polyhedron a bit more and test it more (if not in production) though...
Adds support for new log headers with commit 15660c0cba6babf2d6474af41ea939f48ff2077d Signed-off-by: qaz <dev@malplena.net>
|
I've tested it again with the newest Prism version and noticed some changes in the way things are logged which prevented the extraction of the mod list. QazCetelic/polyhedron@15660c0 makes it also work with the new order. I've updated the Cargo.toml to use that commit. Also, if you have any questions about how "polyhedron" works feel free to send a message on Discord. I also made a simple CLI that you could use if you want to more easily test some things locally. |
Reworked log analysis using log analysis crate as discussed in Discord channel.
All checks can be found here and include tests with example data.
This is a breakdown of how common these are detected in my test data. Overall, it detects something in about 53% of all logs.
