dcs-liberation / dcs_liberation

DCS World dynamic campaign.
GNU Lesser General Public License v3.0
715 stars 185 forks source link

Frontlines on Channel Map extend into the water #2612

Closed Starfire13 closed 1 year ago

Starfire13 commented 1 year ago

Affected versions

Development build

Build information

DCS Liberation 7.0.0 Build 4551 Git revision 3863b8ef4097a7c103834a529a5b6792a9b21913

Description

This bug was first reported on Discord by Revi here: https://discord.com/channels/595702951800995872/595706079459934208/1046461341197664357

Since he hasn't filed it yet, I'll go ahead and do so as I've verified the bug exists.

I had a look at the channel_terrain.miz file in resources\tools\ and it looked fine to me AFAICT.

Map in Liberation Screenshot 2022-12-03 083440

Map in mission editor Screenshot 2022-12-03 083504

Save game and other files

Dunkirk #4551.zip

DanAlbert commented 1 year ago

During channel campaign creation:

2022-12-20 12:37:50,163 :: root :: ERROR :: Failed to load landmap resources\theaters\the channel\landmap.p
Traceback (most recent call last):
  File "C:\Users\dan\src\dcs_liberation\game\theater\landmap.py", line 34, in load_landmap
    return pickle.load(f)
_pickle.UnpicklingError: invalid load key, '\x0c'.

Checked 5.2.1, and this is a regression.

DanAlbert commented 1 year ago
(venv) ~\src\dcs_liberation [fix-front-line-extent]> git show --name-status 18f1048dc4226fbbfb532b30dc6c83508ec66f0f
commit 18f1048dc4226fbbfb532b30dc6c83508ec66f0f
Author: Dan Albert <dan@gingerhq.net>
Date:   Wed Sep 7 16:36:12 2022 -0700

    Migrate The Channel to YAML.

M       game/campaignloader/campaign.py
M       game/theater/conflicttheater.py
D       game/theater/seasonalconditions/thechannel.py
A       resources/theaters/the channel/info.yaml
R100    resources/thechannel.gif        resources/theaters/the channel/landmap.p

Oops. Bad rename.