Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Metoda robota walk nefunguje správně #185

Closed
JureckovaK opened this Issue Mar 5, 2019 · 9 comments

Comments

Projects
3 participants
@JureckovaK
Copy link
Contributor

JureckovaK commented Mar 5, 2019

Když spustím stávající kód, tak v momentě, kdy robot stoupne na díru, spadne hra. Podle chybové hlášky je problém v robotí metodě walk. Narazila jsem na to dnes při spuštění mapy test_effects.json.

@befeleme

This comment has been minimized.

Copy link
Contributor

befeleme commented Mar 5, 2019

Problém je způsoben změnou koordinát neaktivních robotů (-1, -1) na None.

@befeleme

This comment has been minimized.

Copy link
Contributor

befeleme commented Mar 6, 2019

@ivet1987, chceš se na to podívat? Myslím si, že tento debugging bude zajímavý. Skrz chybové výpisy přijdeš na to, kde je třeba ošetřit podmínky pro výskyt None.

@ivet1987

This comment has been minimized.

Copy link
Contributor

ivet1987 commented Mar 6, 2019

Jestli to spěchá tak tento týden bohužel nebudu mít čas :(

@ivet1987

This comment has been minimized.

Copy link
Contributor

ivet1987 commented Mar 15, 2019

@JureckovaK @befeleme můžete mě kopnout jak postupovat? Chybové hlášky vidím, ale nevím co vlastně napsat za podmínky, pokud ty coordinates jsou None, tak co?

@befeleme

This comment has been minimized.

Copy link
Contributor

befeleme commented Mar 17, 2019

@ivet1987, neslibuju, ze se dostanu k pocitaci dalsi dva dny :/
ale: problem je ve funkci

def get_next_coordinates(coordinates, direction):
    """
    Get next coordinates in the given direction from current coordinates.
    """
    (x, y) = coordinates
    (new_x, new_y) = direction.coor_delta
    x = x + new_x
    y = y + new_y
    return (x, y)

Tato funkce bere koordinaty robota, rozdeli je na x, y a vypocita nove na zaklade smeru danemu robotovi. Kdyz jsou koordinaty None, mame problem, nejdou rozdelit, ani nejde k nim nic pricitat.
Bud se tedy zmeni tato funkce, ze nebude nic pocitat, pokud koordinaty jsou None, nebo tam, kde se tato funkce vola, nebudeme do seznamu zahrnovat neaktivni roboty. Nevim, ktera cesta je lepsi, treba ti napovi Katka, pripadne se o tom pobavte v utery na srazu.

@JureckovaK

This comment has been minimized.

Copy link
Contributor Author

JureckovaK commented Mar 19, 2019

No moje první myšlenka byla taková, že když robot stoupne na díru, tak vlastně už nechci, aby se dál hýbal tzn. aby se pohyb robota ukončil. V takovém případě by už nebylo nutné řešit další funkce/metody, které se využívají při pohybu a předpokládají určitý typ vstupu. Podobně jsem to řešila i při aplikaci efektů políček, kdy se v momentě umření robota ukončí provádění konkrétního efektu políčka na konkrétním robotě.

@ivet1987 ivet1987 self-assigned this Mar 19, 2019

@ivet1987

This comment has been minimized.

Copy link
Contributor

ivet1987 commented Mar 19, 2019

tak hra už nepadá ale neprochází testy a já nevím jak je spravit :(
#194

@ivet1987

This comment has been minimized.

Copy link
Contributor

ivet1987 commented Mar 19, 2019

předpokládám protože ty testy volají metodu walk, ve které se smazalo check_hole(nyní fall_into_hole) a ta ubrala robotu život že? Já jen jestli to dobře chápu. Ale nevím jak to opravit.

@JureckovaK JureckovaK added this to Rozdělané in Projekt Mar 20, 2019

@ivet1987

This comment has been minimized.

Copy link
Contributor

ivet1987 commented Mar 21, 2019

@JureckovaK JureckovaK closed this Mar 21, 2019

Projekt automation moved this from Rozdělané to Hotovo Mar 21, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.