valarnin
commented
3 years ago Seems fine to me in emulator at least for the first fight in Tower at Paradigm's Breach in raidemulator. Does it reproduce for you in emulator?
Closed Akurosia closed 3 years ago
valarnin
commented
3 years ago Seems fine to me in emulator at least for the first fight in Tower at Paradigm's Breach in raidemulator. Does it reproduce for you in emulator?
Akurosia
commented
3 years ago Before i forget to mention it, me and my friend have both the game set to DE and also all in cactbot is set to DE. @valarnin Raidemulator is also not working for me in these instances (same results as in game, with e.g. uwu and titan-ex working fine)
valarnin
commented
3 years ago Hmm, can you upload a log, preferably for a shorter fight e.g. titan ex, which isn't working? I can debug from there. Might be something specifically related to DE.
Edit:
Sorry, I meant any shorter fight along the same lengths as Titan Ex, which isn't working for you.
Akurosia
commented
3 years ago apparently i dont have a small fight where it does not work, but can provide both fights where i noticed it (also got a report from one of my friends that its also valid for castrum (bozja))
valarnin
commented
3 years ago Any fight is fine, it's just faster to debug a smaller fight.
On Tue, Jun 8, 2021, 14:52 Akurosia Kamo @.***> wrote:
apparently i dont have a small fight where it does not work, but can provide both fights where i noticed it (also got a report from one of my friends that its also valid for castrum (bozja))
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/quisquous/cactbot/issues/3040#issuecomment-857012637, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABOWBLVWU6J6ZDUSTZNIK5TTRZRHVANCNFSM46JSH6IA .
Akurosia
commented
3 years ago TheTowerAtParadigmsBreach_SperrgebietA_20210608_081504_5m37s.log
this is the first fight of Nier raid 3 (splitted with the cactbot splitting tool)
valarnin
commented
3 years ago /Noch 15 Sekunden, bis sich (?:(?:der|die|das) )?(?:Zugang zu(?:[rm]| den)? )?00:0839:Sperrgebiet A schließt/iThis regex doesn't look right. The line it's attempting to match:
"[04:15:04.0] 00:0839:Noch 15 Sekunden, bis sich das Sperrgebiet A schließt."yes thats not right, but it was working some releases befor
valarnin
commented
3 years ago Trying to bisect the change that caused the problem.
valarnin
commented
3 years ago This is the commit that caused the bug I think: https://github.com/quisquous/cactbot/commit/8bc971c7a632ac42fffe029eb23ec0ea2986277b
valarnin
commented
3 years ago Pretty sure this breaks timeline sync in all languages except en/ja, for any timeline that starts with an area seal message and doesn't have another sync that overlaps relative 0 on the timeline.
valarnin
commented
3 years ago Debugging timeline.ts's GetReplacedHelper function shows:
Is the regex escaped improperly or something?
panicstevenson
commented
3 years ago That commit is from a year ago, this has been broken for a year?
valarnin
commented
3 years ago That commit is from a year ago, this has been broken for a year?
Sorry, it was actually this commit. Thought I had updated my comment but apparently not.
https://github.com/quisquous/cactbot/commit/0f02b84a3fde14e83f3cc7a2d44479ff3f8b807b
In the last few days i noticed that for some contents (e.g. Delubrum normal, Nier raid Nr.3) i do not have timelines anymore (alerts are working fine). In other content (e.g. Uwu, Titan Ex, test timelines in limsa) its working fine
For myself i am using the latest nightly build (self compiled) and a friend of mine uses the latest official distributed version (upgraded through ACT itself)
Following is the Overlayplugin output: