Jimm1432 / Rush-Royale-Bot

Python based bot for Rush Royale
MIT License
13 stars 4 forks source link

MODIFIED BOT_CORE.PY #15

Closed hatehackppl closed 8 months ago

hatehackppl commented 9 months ago

Hi, i have modified some commands code in Bot_Core.py for Coop support unit purpose (Shaman), how to apply into the bot as i did press saved but the bot still not follow my code...

I modified the code to priority copy shaman unit with harleyquin instead of copy itself.

Thanks for your reply in advanced....

image

Jimm1432 commented 8 months ago

You would have to add the shaman to the series list somewhere else in the code. Changing the unit there will only select it if it's already in the special merge series.

hatehackppl commented 8 months ago

You would have to add the shaman to the series list somewhere else in the code. Changing the unit there will only select it if it's already in the special merge series.

Hi Jim, can you advise which line have to add shaman in the code please... i tried to modified the code in special merge but harlequin still not copying shaman

Jimm1432 commented 8 months ago

What units are you using in the deck?

hatehackppl commented 7 months ago

What units are you using in the deck?

Support decks... (Shaman, cauldron, harlequin, witch, demonologist) Possible to let the deck only copy shaman? i run bot more on PVE only

Jimm1432 commented 7 months ago

If you have shaman as the DPS unit it should work. It will try to keep the cauldrons at 4 only.

If I'm honest the whole script needs replacing and keeping more simple unit merging.

On Fri, 2 Feb 2024, 00:47 hatehackppl, @.***> wrote:

What units are you using in the deck?

Support decks... (Shaman, cauldron, harlequin, witch, demonologist) Possible to let the deck only copy shaman? i run bot more on PVE only

— Reply to this email directly, view it on GitHub https://github.com/Jimm1432/Rush-Royale-Bot/issues/15#issuecomment-1922584400, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA . You are receiving this because you modified the open/close state.Message ID: @.***>

hatehackppl commented 7 months ago

If you have shaman as the DPS unit it should work. It will try to keep the cauldrons at 4 only. If I'm honest the whole script needs replacing and keeping more simple unit merging. On Fri, 2 Feb 2024, 00:47 hatehackppl, @.> wrote: What units are you using in the deck? Support decks... (Shaman, cauldron, harlequin, witch, demonologist) Possible to let the deck only copy shaman? i run bot more on PVE only — Reply to this email directly, view it on GitHub <#15 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA . You are receiving this because you modified the open/close state.Message ID: @.>

I tried to use back clean script and set shaman as DPS. Unfortunately Harleyquin still not copying shaman, it only merge itself. I think have to modify some script in core.py to let harleyquin copy only shaman. But i have no idea which section which line i have to modify honestly...

Jimm1432 commented 7 months ago

You need to add it to the special merge list, I'll have a look and see if I can make you a core.py for your deck but it won't work for any other decks

On Fri, 2 Feb 2024, 23:53 hatehackppl, @.***> wrote:

If you have shaman as the DPS unit it should work. It will try to keep the cauldrons at 4 only. If I'm honest the whole script needs replacing and keeping more simple unit merging. … <#m1734796800078743494> On Fri, 2 Feb 2024, 00:47 hatehackppl, @.> wrote: What units are you using in the deck? Support decks... (Shaman, cauldron, harlequin, witch, demonologist) Possible to let the deck only copy shaman? i run bot more on PVE only — Reply to this email directly, view it on GitHub <#15 (comment) https://github.com/Jimm1432/Rush-Royale-Bot/issues/15#issuecomment-1922584400>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA . You are receiving this because you modified the open/close state.Message ID: @.>

I tried to use back clean script and set shaman as DPS. Unfortunately Harleyquin still not copying shaman, it only merge itself. I think have to modify some script in core.py to let harleyquin copy only shaman. But i have no idea which section which line i have to modify honestly...

— Reply to this email directly, view it on GitHub https://github.com/Jimm1432/Rush-Royale-Bot/issues/15#issuecomment-1924915864, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI2CHWJXGCP4ERLUWHQ37N3YRV36VAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRUHEYTKOBWGQ . You are receiving this because you modified the open/close state.Message ID: @.***>

hatehackppl commented 7 months ago

You need to add it to the special merge list, I'll have a look and see if I can make you a core.py for your deck but it won't work for any other decks On Fri, 2 Feb 2024, 23:53 hatehackppl, @.> wrote: If you have shaman as the DPS unit it should work. It will try to keep the cauldrons at 4 only. If I'm honest the whole script needs replacing and keeping more simple unit merging. … <#m1734796800078743494> On Fri, 2 Feb 2024, 00:47 hatehackppl, @.> wrote: What units are you using in the deck? Support decks... (Shaman, cauldron, harlequin, witch, demonologist) Possible to let the deck only copy shaman? i run bot more on PVE only — Reply to this email directly, view it on GitHub <#15 (comment) <#15 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA https://github.com/notifications/unsubscribe-auth/AI2CHWOM7TFBEMMY7JCASODYRQZQXAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRSGU4DINBQGA . You are receiving this because you modified the open/close state.Message ID: @.> I tried to use back clean script and set shaman as DPS. Unfortunately Harleyquin still not copying shaman, it only merge itself. I think have to modify some script in core.py to let harleyquin copy only shaman. But i have no idea which section which line i have to modify honestly... — Reply to this email directly, view it on GitHub <#15 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AI2CHWJXGCP4ERLUWHQ37N3YRV36VAVCNFSM6AAAAABBD47IYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRUHEYTKOBWGQ . You are receiving this because you modified the open/close state.Message ID: @.>

Thank you Jim, much appreciated

hatehackppl commented 6 months ago

@Jimm1432 sorry for bothering, im still not able to find the option to let harlyquin copy shaman code. could you help please

Jimm1432 commented 3 months ago

Sorry for the very late reply I just haven't had the time to be doing things with the computer, a simple solution would be to just rename the shaman icon to demon_hunter in all units. Then add it as DPS unit in your profile. You might need to edit so it will merge at rank 4+ and not start saving them.

One day soon I want to totally scrap the current unit merge lists and streamline it somehow, it's so messy and slow. I'll be updating for the premium upgrade, it will claim the ads and buy the cards. If I can get it working correctly I'll add using sands of time with a lose in clan tournament.

ToxicGigante commented 2 months ago

You are the best!