Can't get field 'unit_ids' for 'move' action

[ElKrist]

Steps to reproduce:

• Recorded game used (I've tried with many others and there is the same problem): https://www.voobly.com/files/view/53123676/qnc2045gpo3x3vkyyqks5h526pu725er

• Code used:

import sys
from mgz import header, body
import pprint

if __name__ == "__main__":
    pp = pprint.PrettyPrinter(indent=4)
    # For each input filename
    for arg in sys.argv[1:]:
        with open(arg, 'rb') as f:
            # Remember end of file
            f.seek(0, 2)
            eof = f.tell()
            f.seek(0)
            # Parse the header
            h = header.parse_stream(f)
            # Parse the body
            while f.tell() < eof:
                # Parse a body operation
                o = body.operation.parse_stream(f)
                if o.type == 'action' and o.action.type == 'move':
                    pp.pprint(o)

Problem:
None of the 'move' action has the field 'unit_ids'

I see that this field should be present (excepted if moving the same unit as previous command):
https://github.com/happyleavesaoc/aoc-mgz/blob/master/mgz/body/actions.py#L63

I'm not fluent in python so please forgive me if I made a simple mistake

Thanks for your work!

[ElKrist]

I've found a "trick" to make it work.
I replaced (error, see comments below):

If(lambda ctx: ctx.selected > 0x01, Array(
    lambda ctx: ctx.selected, "unit_ids"/Int32ul
))

by

    "unit_ids"/If(lambda ctx: ctx.selected < 0xff, Array(lambda ctx: ctx.selected, Int32ul))

If selected < 255 it gives for example: 'unit_ids': [1910]
When selected=255 it gives 'unit_ids': None

[happyleavesaoc]

Hi, do you mean ai_move or move?

[ElKrist]

move

[happyleavesaoc]

I'm looking at the code and it already uses selected < 0xff: https://github.com/happyleavesaoc/aoc-mgz/blob/master/mgz/body/actions.py#L63

[ElKrist]

Oh sorry when I said I replaced that code:

If(lambda ctx: ctx.selected > 0x01, Array(
    lambda ctx: ctx.selected, "unit_ids"/Int32ul
))

I made a mistake (the code above does not exist). I meant I replaced this code:

    If(lambda ctx: ctx.selected < 0xff, Array(
        lambda ctx: ctx.selected, "unit_ids"/Int32ul
))

The trick was to add "unit_ids"/ in front of the If

[happyleavesaoc]

Fixed in 247c246