Opened 10 years ago

Closed 10 years ago

#2792 closed defect (fixed)

Petra js errors

Reported by: serveurix Owned by:
Priority: Should Have Milestone: Alpha 17
Component: Core engine Keywords: petra, javascript, errors
Cc: Patch:

Description

I get a flood of js errors when playing Arcadia on svn, r15728 I think. Those errors make the game lag terribly.

Here are the *log.html files.

Attachments (3)

lastlogs.tar.xz (940 bytes ) - added by serveurix 10 years ago.
commands.txt.xz (8.4 KB ) - added by serveurix 10 years ago.
corral.diff (1.7 KB ) - added by mimo 10 years ago.

Download all attachments as: .zip

Change History (10)

by serveurix, 10 years ago

Attachment: lastlogs.tar.xz added

comment:1 by mimo, 10 years ago

Do you have an unmodified version of the game ? in your log files, the first error I see is from UnitAI: ERROR: Animal got moved into INDIVIDUAL.* state it should never happen, and I've never seen it in any games. If yes, could you upload your commands.txt file because I cannot reproduce your problem.

in reply to:  1 comment:2 by serveurix, 10 years ago

Replying to mimo:

Do you have an unmodified version of the game ? in your log files, the first error I see is from UnitAI: ERROR: Animal got moved into INDIVIDUAL.* state it should never happen, and I've never seen it in any games. If yes, could you upload your commands.txt file because I cannot reproduce your problem.

This error happens when I ring the city alert and some sheep are created at the corrals while the alert is on. I'll try to find the corresponding commands.txt.

by serveurix, 10 years ago

Attachment: commands.txt.xz added

comment:3 by serveurix, 10 years ago

This is the good commands.txt I think.

by mimo, 10 years ago

Attachment: corral.diff added

comment:4 by mimo, 10 years ago

Thanks. I'll have a look at it. For the corral bug, I attach here a patch. In fact, none of the units in production were tested on the alert level and were made automatically in alert. I've changed that behaviour. And to correct the corral bug, animals are set to an alert level of 99 so that they are never put on alert.

comment:5 by mimo, 10 years ago

The other errors you get are due to inconsistencies in the techs following the merge of balancing branch: pair_champ_02.json refers to the tech attack_citizensoldiers_will which does not exist. This tech should in fact be attack_soldiers_will.

comment:6 by scythetwirler, 10 years ago

In 15738:

Fixes #2795. Refs #2792

comment:7 by leper, 10 years ago

Milestone: BacklogAlpha 17
Resolution: fixed
Status: newclosed

With r15736 this should be fixed now.

Note: See TracTickets for help on using tickets.