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

raidemulator can't reset battles correctly #5714

Open
choyike opened this issue Jul 20, 2023 · 8 comments
Open

raidemulator can't reset battles correctly #5714

choyike opened this issue Jul 20, 2023 · 8 comments
Labels

Comments

@choyike
Copy link

choyike commented Jul 20, 2023

I'm trying out use the raidemulator to write my own scripts. It will work fine until July 10th. When I tried to load logs yesterday, a logs file only read first fight fine, it couldn't read other fights under the same log file correctly anymore. When the battle is wipe, it reports the following error. My logs files(include old files) have same error.
bug

By the way,I have tried masking my own scripts file,not using OVERLAY_WS to load raidemulator(using the url:https://quisquous.github.io/cactbot/ui/raidboss/raidemulator.html), clear the browser cache, use another network and so on,
it has the same error.

I try to split the logs into one file has only one fight, the raidemulator can read the file correctly.

@choyike choyike added the bug label Jul 20, 2023
@Souma-Sumire
Copy link
Contributor

Souma-Sumire commented Sep 2, 2023

I also encountered the same issue, and I've been trying to pinpoint the problem. Currently, I've noticed that this issue started occurring after c245ef6. However, I haven't made any progress on how to fix it. It seems like the Emulator's team initialization logic doesn't fully adapt to the new State update logic.

The code interrupts in the updateState function of AnalyzedEncounter.ts due to the inability to correctly retrieve the job.

I believe that the actual data parsing currently occurs after the load logic in the Raid Emulator, including ID and job information. This is causing the initialization of the party list to fail.

The issue only exists in non-first encounters.

@Souma-Sumire
Copy link
Contributor

Souma-Sumire commented Sep 3, 2023

Perhaps it's an issue with encounter segmentation, but I'm not entirely sure. Here is the partyList log:

  • 11|2023-08-30T21:02:53.0760000+08:00|8|1034xxxx|102Fxxxx|1031xxxx|102Fxxxx|1029xxxx|102Exxxx|102Bxxxx|102Cxxxx|d8ba31f23845f110
  • 11|2023-08-30T21:06:26.0640000+08:00|8|1034xxxx|1031xxxx|102Fxxxx|1029xxxx|102Exxxx|102Bxxxx|102Cxxxx|cd6d7cff39b8e2f2
  • 11|2023-08-30T21:08:07.2590000+08:00|8|1034xxxx|102Fxxxx|1031xxxx|102Fxxxx|1029xxxx|102Exxxx|102Bxxxx|102Cxxxx|a08937fe180a9eea
  • 11|2023-08-30T21:45:21.0270000+08:00|0|651c08c365944360

At 21:06, it seems like someone disconnected or something happened. Although the count is 8, there are only 7 IDs. I'm not sure if this is the reason?

This log was also given to me by someone else, so I don't know exactly what happened at that time.

This is the segmented encounter:
image

@Souma-Sumire
Copy link
Contributor

Souma-Sumire commented Sep 3, 2023

It seems to be caused by being split by the fflogs uploader.

edit: It seems not, even complete logs can cause problems.

@Souma-Sumire
Copy link
Contributor

Souma-Sumire commented Sep 13, 2023

I have been deeply troubled by this issue, as a large number of battle records were unusable. Therefore, I have been trying to figure out how to fix this problem.

Now, I have made some progress.

With this commit (edited), I can now successfully read battle records.

@quisquous
Copy link
Owner

Hmm! What is the bug here, ultimately? Is it that nextState(0) can't find a state that nextState(encStart) can? That is kind of surprising to me.

(cc @valarnin)

@quisquous
Copy link
Owner

@valarnin This happens with the 2nd and 3rd pulls in the log (and others) of the log you sent me.

@valarnin
Copy link
Collaborator

valarnin commented Dec 2, 2023

Yeah, this is on my list of things to look into but it's a bit far down the list since it's resolved with a refresh when it happens.

@quisquous
Copy link
Owner

For what it's worth, that's not true for me. The 2nd and 3rd pulls in the log you sent me always have this error, regardless of refreshes or db clears and reloads.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants