Opened 3 years ago

Last modified 9 months ago

#6201 new defect

Replay filter broken — at Version 7

Reported by: Langbart Owned by:
Priority: Should Have Milestone: Alpha 27
Component: UI & Simulation Keywords: regression
Cc: Patch:

Description (last modified by Langbart)

When checking the replay screen in the latest SVN version [25658], the Filter was enabled, but there were still some gray replays displayed.

I will attach the my replay folder as a zip file - 0.0.25.zip

A short discussion about the problem can be found in IRC 0ad-dev (3/Jun/21)

18:28 < Angen> Langbart: yeah, something is wrong with filter
18:29 < Angen> it sometimes does not filter some replays
18:29 < Angen> and sometimes it filters all

Edit 1: The problem probably starts with changeset [25634], but errors are displayed until it is fixed in [25651]. This means that the problem is probably related to [25634], but you can only check it with [25651] for the first time.

Change History (10)

by Langbart, 3 years ago

Attachment: replay_screen.jpg added

by Langbart, 3 years ago

Attachment: 0.0.25.zip added

comment:1 by Imarok, 3 years ago

Hmm, I can't reproduce the issue with your data. Did you have mods enabled?

in reply to:  1 comment:2 by Langbart, 3 years ago

Replying to Imarok:

Hmm, I can't reproduce the issue with your data. Did you have mods enabled?

No, the problem is still there with the latest SVN version [25758], but the replay filter works perfectly with the recently released macOS bundle for A25.

PS: I used the same replay folder to test the macOS bundle and the SVN version.

comment:3 by Freagarach, 3 years ago

It may or may not be related, but when I return from a replay it doesn't show any replays at all, unless I untick and tick the filter. r25763

comment:4 by wraitii, 3 years ago

can't reproduce any issue.

in reply to:  3 ; comment:5 by Imarok, 3 years ago

Replying to Freagarach:

It may or may not be related, but when I return from a replay it doesn't show any replays at all, unless I untick and tick the filter. r25763

I also can't reproduce that. I guess that means that you have to investigate the code :/

in reply to:  5 comment:6 by Langbart, 3 years ago

Replying to Imarok:

I guess that means that you have to investigate the code :/

Good conversation on IRC 0ad-dev (11/Jul/21) between me and Stan, the problem boils down to the changes from changeset 25634 (2/Jun/21) with the introduction of ignoreInCompatibilityChecks.

comment:7 by Langbart, 3 years ago

Description: modified (diff)

Additional observations added to the description for changeset [25633], [25634] and [25651].

by Langbart, 3 years ago

Attachment: 25651_25634_25633.jpg added
Note: See TracTickets for help on using tickets.