Closed therippe closed 10 years ago
Hello,
Alors quelques commentaires constructifs :)
Après niveau détails :
Pour simplifier tes modifs je te conseille de faire un git squash (ca va compiler tes 3 commit en 1 seul) puis pour tes corrections de faire un commit mais en mode amend pour que ca édite le commit en cours et après tu push en mode force, comme ca ton fork est mis à jour et le PR ici reflète automatiquement les modifications
merci pour cette réponse je reprend tout çà ce soir ou demain :) du coup je dois faire quelque chose pour la demande en cour ?
ou je repars de la version 1.6 et je fais mes modifs "propre" selon tes conseils ?
Tu peut merge la 1.6 dans ta branche cela devrait garder tes modifs.
Pour la demande en cours comme je t'ai dis si tu met a jour ton fork ca met a jour ici aussi.
Regarde la doc pour les commandes dont j'ai parler a la fin de mon commentaire (Squash / amend et push --force)
tout çà ne peut se faire qu'en ligne de commande non ? la version windows ne propose pas çà si ?
bon je repars a zero ^^
Le client github windows est juste pourri :)
Utilise un truc un peu plus avancer comme http://www.collab.net/giteyeapp ou tortoisegit :)
Sinon, t'as dans le client github le moyen de lancer un shell (invite de commande). Puis pour merger le master dans ton fork, il faut suivre ça : https://help.github.com/articles/syncing-a-fork
je suis reparti à zero avec la version 1.7.5 histoire d'avoir un truc propre ^^ trop peur de faire des conneries
Sinon j'ai suivi ton conseil et renommé en pauseafterbounty avec le message popup qui va bien
pour la partie bountyandsequence comment detecter si c'est dejà fait ?
vu qu'on utilise des séquences de type "Act1_BOUNTY"
et
d'autres "Act1_ADV" pour le mode aventure ?
vue que ces modifs sont appelés à fonctionner de pair à savoir faire les bounty connus, puis de chainer la sequence aventure ET d'attendre l'intervention humaine pour finir les bounty on réduit le risque de détection non ?
Eventuellement je peux conditionner l'utilisation de BountyAndSequence à PauseAfterBounty à True pour éviter ce risque ?
j'attend votre retour pour envoyer mes modifs :)
EndSequenceOnBountyCompletion y a pas eu une evolution ? car la la des qu'il est dans une de mes sequences hors bounty, il va dans la zone et hop passse à la suivante
(celà dit c'est pas que dans mon cas, car pour certaines bounty "officielles" il arrive dans la zone qui n'est pas celle du bounty (mais bien celle de la sequence correspondante, il va changer de zone "a pied" ) et du coup l'abandonne
il me semble que précédemment il fallait qu'il y ait le pop de la box "bounty complete" pour qu'il considere cela finit (du coup si on avait interrompu avec F3 et fini à la main, il ne "savait" pas que c'etait fini) alors que la c'est un autre mode de test non ?
A priori du à la modif d'ano sur le fichier sequence :) "Fix la détection de la quete en cours quand on commence un bounty " non ?
Heu non il a jamais attendu la box c'est par détection pareil, après y'a peu être un bug mais comme d'hab need logs :)
Pour ton autre message, comme je t'ai dis plus tot sans modification interne profonde on peut pas couvrir les cas des séquences différentes, mais il y a bcp de bounty ou on utilise des map ADV.
Donc ce que je t'avais proposé en gros c'était de ne pas ajouter les map aventures qu'on a déjà faites en bounty (simple comparaison).
Ca couvre pas 100% des cas mais au moins 60 a 70% des cas et c'est bcp mieux que rien.
ok j'ai fait comme tu me l'as conseillé, je n'ajoute que les séquences qui n'ont pas été auto générées,
j'ai mis ton fix 1.7.6 et du coup c'est ok vu que çà abandonnait systématiquement les séquences pour lesquelles il n'y avait pas de quête.
petit amusement j'ai rajouté un envoi de mail a la mise en pause du bot pour m'avertir de venir finir les bountys ^^
A priori t'as pas push :)
Pour l'envoi de mail je suis pas trop pour, c'est ultra spécifique c'est du code a maintenir et de la complexification pour quasi 0 utilité.
merci kyria j'ai fait comme tu m'as dit pour sync les branches. j'ai donc refait un pull request (j ai pas mis l'histoire du mail, c'etait juste pour tester personnelement, faut que je me fasse un envoi de sms la prochaine fois ^^) par contre il y a des diff par rapport a des tabulations, je ne sais pas trop d'où çà sort pourtant j'ai pas fait de mises en pages, j'imagine que çà doit venir de mon editeur ?
ajout en 3 commit .. le temps de comprendre le bon fonctionnement du sync :)
Stop bot apres bounty et/ou chaine la sequence aventure
Ajout de deux variantes globales + ini afin de pouvoir arreter le bot a la fin de la sequence et finir les bounty manuellement. possibilité d'enchainer la sequenceaventure apres la sequence automatique bounty pour obtenir un run plus long