-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Added AI Flags Documentation To Directory For New Documentation Site #5057
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add somewhere at the top that generally it is recommended to use AI_FLAG_CHECK_BAD_MOVE
, AI_FLAG_CHECK_VIABILITY
, AI_FLAG_TRY_TO_FAINT
for good AI. Everything else should be used with consideration
Thanks for the changes. I'll wait for @Pawkkie for feedback before merging |
This looks good to me, I'll backport the changes to the wiki page once this gets merged. My only piece of feedback is that I'm not sure "Keep in mind that this is a weaker form of If we want a comparative clause adding to that section, I might suggest something like "Keep in mind that this is weaker than AI_FLAG_TRY_TO_FAINT at scoring OHKOs" instead. @AlexOn1ine thoughts? |
Yeah, I like this suggestion |
No problem I'll make the changes in the morning and also setup the new docs site on my local machine and create a new PR to add it there as well. |
That should be all the requested changes. Let me know if there's anything else. I have also started moving over the other documentation from the wiki which will be in a future PR. |
Description
This pull request adds the file
ai_flags.md
to the/docs
directory for the new documentation website.Future contributions
I look forward to contributing to more documentation. Please let me know if this is the correct directory to add documentation to in the future.
Discord Contact Info
toonv#26526