Skip to content

Conversation

Disatiler
Copy link
Contributor

Bonjour Monsieur,

Voici le code de Laby avec maintenant l'apparition des traces de pas de la fourmi ainsi que la superposition d'image.
Cependant aucun test par rapport à l'empilement n'a été fait car les fonctions ne sont pas de Boolean et donc un Assert n'est peut-être pas le plus pertinent pour tester les fonctions.
De plus, lors de l'implémentation de mon code dans ce dossier, j'ai du faire quelque modification de chemin et donc le sprite du cailloux ne veut pas être affiché correctement lorsque la fourmi en porte une! (J'essaye de réparer ce problème au plus vite!!)
Il y a aussi d'autres informations importante au sein même du code d'origine qu'il faudra clarifier pour obtenir un projet parfait!

@nthiery
Copy link
Owner

nthiery commented Jun 30, 2020 via email

@Disatiler
Copy link
Contributor Author

Fin de test ajouté et appliqué dans le main.

Le bugg vient du chemin où il doit chercher le sprite à superposer. L'explication sera plus précise demain.

Par contre je n'ai aucune idée de comment spliter mon projet pour vous le donner en deux Pull request les mis-à-jour du projet.

@Disatiler Disatiler changed the title Ajout trace et empillement de sprite Ajout trace Uniquement Jul 3, 2020
@Disatiler
Copy link
Contributor Author

Bonjour,

J'ai modifié la branche master pour qu'il n'y ai que l'ajout des traces de pas dans le code.
Le pull request sur l'empilement des sprites ne devraient pas tarder non plus!

@@ -0,0 +1,208 @@
{
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ce notebook n'est pas lié à cette PR. À supprimer.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Malheureusement pour lui. Modification faite!

Copy link
Contributor Author

@Disatiler Disatiler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voici les modifications que j'ai faites.
Il m'en reste deux ou trois mais j'aimerais bien qu'on en discute ensemble dessus pour éviter toute confusions.

Comment on lines 445 to 447
bool sow() {
Tile tile = board.get(position);
if ( tile == Tile::Web or
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne comprends pas cette modification que vous voulez apporter

@@ -0,0 +1,208 @@
{
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Malheureusement pour lui. Modification faite!

@Disatiler
Copy link
Contributor Author

Normalement tout est bon, les seuls modifications restant sont l'essentiel!

@Disatiler
Copy link
Contributor Author

Bonjour Monsieur!

J'ai fait les dernières modifications nécessaires pour les traces de pas, normal tout ce dont nous avons parler est présent là-dedans.

Seulement, j'ai un petit soucis de taille, c'est que lorsque je veux appeller les nouvelles fonctions "trace_de_pas()" et "traces_de_pas(bool flag)" mais celle-ci donne interpretor error malgré leur existance dans le global_fr...

@Disatiler
Copy link
Contributor Author

Dernière Mise-à-jour (du moins je l'espère) pour les traces de pas dans Laby!!
Prochaine étape les intégrer au solveur une fois que la Pull Request sera mergé!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants