Mismatch between high-level and low-level actions after preprocessing

[RavenKiller]

Hello!

I'm new to ALFRED and trying to learn it. After reading #84, I find there is still a mismatch between high-level and low-level actions after preprocessing. For example, in pick_two_obj_and_place-AppleSliced-None-CounterTop-10/trial_T20190907_061009_396474, the generated low_to_high_idx is [..., 10, 11, 11, 13]. However, 13 is out of the bound of instr list and also exceeds the maximum high_idx in action_high (which is 12).

I'm not sure if it is a bug because this phenomenon does not cause problems when training baseline models. I think the reason is this line:

alfred/data/preprocess.py

Line 212 in 1898b83

conv['num']['action_low'][-1][0]['high_idx'] = len(conv['plan']['high_pddl']) - 1

which may be better like this:

conv['num']['action_low'][-1][0]['high_idx'] = conv['plan']['high_pddl'][-1]['high_idx'] - 1

[MohitShridhar]

@RavenKiller, sorry for the late reply.

I'll look into this when I get some time. But for now, yeah, if you visualize those high-level actions, and it makes sense to merge/discard them, then use your fix.