search

dice4321 / Gunkolle

This script was originally meant to execute kancolle expeditions but was repurpose for girls frontline (by Ryuuhou https://github.com/Ryuuhou/AHKanColle)
Other
16 stars 14 forks source link

Enables two DPS dragging #56

Closed fnzr closed 5 years ago

fnzr commented 5 years ago

These changes allow to optionally use 2 loaded dolls on drag runs To use 2 pairs of dolls, add Doll3 and Doll4 to the config.ini:

Doll1=M4A1
Doll2=M4SOPMODII
Doll3=G11
Doll4=AR15

Doll 1 is paired with Doll 2 and Doll 3 with Doll 4. In this example, at every run M4A1 will swap with Sop and G11 with AR15. To disable this feature, simply don't add Doll3/4 entries or comment them out with # To allow a corpse to be the echelon leader, I also increased the formation search to the third slot.

dice4321 commented 5 years ago

This is an excellent feature, along with the drag you have shown us, could possibly pave way to boss maps and such. I see you have commented out 

;tpc := 0
;pc := []
;pc := [FormationProfile]
;tpc := WaitForPixelColor(FormationProfilex,FormationProfiley,pc)

Was this giving you issues? I find that, the longer the script runs, the longer the animations take, and WaitForPixelColor() waits for the Tdoll portraits to appear after before running WFindClick() to find the doll. I can impliment another way of waiting for the portraits that doesnt use WaitForPixelColor() if it is an issue.

fnzr commented 5 years ago

I meant to bring that up but it slipped my mind. Yes, right on the first formation swap it would get stuck in that line. Since it worked without it, I just commented out and forgot about it. This check is something we should have though. I hamfisted a sleep 500 after if dualDPS, but your check seems more reasonable. You could try using WaitForPixelColor without changing the implementation. Maybe I was using it wrong and that's way it got stuck.