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

Adding a new updated bot for Voiceless Voice #193

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

owenshi2
Copy link

@owenshi2 owenshi2 commented May 4, 2024

My attempt at programming a modern deck. Checked the documentation but had some issues with running windbot after adding the executors. Please let me know if there's any errors with the formatting or bot itself encountered as I haven't been able to run or test the bot locally.

@mercury233
Copy link
Collaborator

mercury233 commented May 11, 2024

As you can see, the codes you posted are incomplete.

@owenshi2
Copy link
Author

Thank you for the fixes, mercury! Is this version able to be run in the bot? If so, do I copy the files from the repo into the EDOPro Windbot folder to test it?

@Wind2009-Louse
Copy link
Contributor

There's still many errors in scripts. Even if grammar errors are fixed, there are still many cards that cannot be activated. Like:
Prayers of the Voiceless Voice
Trias Hierarchia
and so on

@owenshi2
Copy link
Author

Hi Louse,

Thank you for taking a look at my code! How do you suggest I correct the issues brought up?
If I remove the cards mentioned from being able to be activated altogether, would the bot figure it out when given the prompt?

@Wind2009-Louse
Copy link
Contributor

Wind2009-Louse commented May 31, 2024

Hi Louse,

Thank you for taking a look at my code! How do you suggest I correct the issues brought up? If I remove the cards mentioned from being able to be activated altogether, would the bot figure it out when given the prompt?

You can check https://github.com/IceYGO/windbot/wiki/How-to-create-your-WindBot-AI.
Bot will activate a card only when you defind it with AddExecutor(ExecutorType.Activate, _id[, func]), or with AddExecutor(ExecutorType.Activate[, func])(if you use the latter one, bot will activate any card it can do, so it's not recommended).

owenshi2 added 2 commits May 31, 2024 15:52
Fixed some cards not activatable.
Got rid of IP Masquerena since the script would be too complicated to implement for me. Perhaps in the future?
@owenshi2
Copy link
Author

owenshi2 commented May 31, 2024

Ok; I checked which cards don't have an activated and added some functions for the unique ones. Do let me know if there's other things to be fixed!

(I also really liked how in-depth the activation functions for SP and Chaos Angel from the Lab Executor you made, @Wind2009-Louse, so I added some of the functions in as well; hope it's alright)

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.

3 participants