how to relocate terran flyable building for a proper addon

[ceacar]

I wrote some code below, doesn't work well

if self.units(UnitTypeId.FACTORY).ready.exists:
    fac = self.units(UnitTypeId.FACTORY).ready.first
    if fac:
        await self.do(fac(AbilityId.LIFT))
        print("try moving to landing point:", fac.add_on_land_position)
        # await self.do(fac(AbilityId.MOVE, fac.add_on_land_position))
        await self.do(fac(AbilityId.LAND, fac.add_on_land_position))

does any one have a solution

[tweakimp]

you are trying to lift and land at the same frame when a factory is ready. a landed factory does not have the ability land you have to lift, and then check for flying factory in the next frame. then make it land

[ceacar]

thanks lot; i m posting my code below in case anybody need it

async def relocate_building_for_addon(self, building_type:'UnitTypeId.BARRACKS', flying_building_type:'UnitTypeId.BARRACKSFLYING'):
    for unit in self.units(building_type).ready:
        if unit.add_on_tag == 0:
            await self.do(unit(AbilityId.LIFT))

    for unit in self.units(flying_building_type).ready:
        await self.do(unit(AbilityId.LAND, unit.add_on_land_position))
        await asyncio.sleep(5)