aboutsummaryrefslogtreecommitdiffstats
path: root/bot (follow)
Commit message (Collapse)AuthorAgeLines
* Continue work on hex and rgb color commandsGravatar bradtimmis2021-09-19-18/+69
|
* Reword json file variables and mappingGravatar bradtimmis2021-09-19-5/+5
|
* Fix Flake8 spacing errorsGravatar brad90four2021-09-15-8/+8
|
* Load json file onceGravatar bradtimmis2021-09-12-5/+4
|
* Merge branch 'main' of https://github.com/python-discord/sir-lancebot into ↵Gravatar bradtimmis2021-09-11-1/+1
|\ | | | | | | color-677
| * Added Hanukka command in November month.Gravatar NIRDERIi2021-09-08-1/+1
| | | | | | This is a solution for the #862 issue, which says the Hanukka holiday can also be in November and not only in December.
* | Remove placeholder commentGravatar bradtimmis2021-09-11-2/+0
| |
* | Add fuzzy match functionGravatar bradtimmis2021-09-11-8/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I made a few changes, the biggest being the fuzzy match function to return a hex color code based on an input color name. Open items that I can think of so far: -Since the json file has color names and hex values, in order to use fuzzy matching for a color name the color must first be converted to hex. Currently there is only a rgb to anything function which returns values in a dictionary. -The main embed creation references the rgb_color before it is defined, should the command function be moved to the bottom of the file or just the main embed creation and sending? -When using the rgb mode, should the user be forced to do (r, g, b) or should the command handle an input of "r, g, b"? If you are reading this, thank you.
* | Merge branch 'color-677' of github.com:brad90four/sir-lancebot into ↵Gravatar bradtimmis2021-09-11-35/+101
|\ \ | | | | | | | | | color-677777777
| * | Added "colour information" and "colour conversion" featuresGravatar CyberCitizen012021-09-11-34/+100
| | | | | | | | | | | | | | | Details: https://github.com/python-discord/sir-lancebot/issues/677 NOTE: get_color_fields (line 122) method explicity requires a valid tuple of RGB values.
| * | Test to capture all user_inputGravatar brad90four2021-09-10-1/+1
| | |
* | | Move .json file to correct folderGravatar bradtimmis2021-09-11-0/+0
|/ /
* | Minor fixesGravatar brad90four2021-09-08-10/+5
| |
* | Merge branch 'color-677' of https://github.com/brad90four/sir-lancebot into ↵Gravatar brad90four2021-09-07-217/+299
|\ \ | | | | | | | | | color-677
| * | Merge branch 'main' into color-677Gravatar brad90four2021-09-07-217/+299
| |\|
| | * Wikiguess Game (#618)Gravatar Shivansh-0072021-09-07-217/+299
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * (trivia-quiz): Add Wikipedia Guess Game This commit also moves all the 'dynamic' question generator to a separate class. Closes: #446 * (trivia quiz): Use tuples for command aliases * (trivia quiz): Edit congratulations message * (trivia quiz): Use default dict for storing player scores * (trivia quiz): 'done_question' to 'done_questions' * Add space after 'Congratulations' word * Use classmethods for dynamically generating questions * Don't add wiki category if max error fetches hit If the task hit max error fetches, which is 3 currently, it would remove wikipedia from listed categories and not add it to loaded questions. If it doesn't hit max fetches, then it adds them. * Don't hardcode the number of questions in RULES * Add information field only if it exists * Add "cs" and "python" categories to the `.quiz` command * add 30 questions each under the categories "cs" and "python" add the two categories into the code and modify the starting phase Co-authored-by: Xithrius <[email protected]> Co-authored-by: ToxicKidz <[email protected]> * refactor: Use yesterday's most read to make trivia questions Since random wikipedia article guess questions weren't really "knowledgeable", no one could really guess it or gain any "good" knowledge from them, so after asking wookie (this commits mentions his review comments above also), I decided to use these. * refactor: Logic to remove pronounciations from question Co-authored-by: wookie184 <[email protected]> * fix: Set to correct question limit If the number of questions are less then the default limit which can happen in the case of wikipedia guess game as it is dependent on the most read articiles on wikipedia, it would create a infinite loop sending us into infinite amount of errors, so let's prevent that, thanks wookie * chore: Add comment for d5f8205 change * refactor: Remove double mention of dataclass in quizentry * chore: Use r"" over noqa We can use r"", a raw string, here to make it clear that \* and \s aren't supposed to be handled as escape sequences and just use the "raw string". * fix: Correct off by one bug Originally, before this commit, we checked the number of questions left by comparing `len(done_questions) > self.question_limit`, so question limit had to be 1 since if it wasn't we would compare 7 > 7, which would be false and then it would send another question. To correct this bug, we now use == comparision on the two, so if the number of done questions is same as the question limit it means that the round is over. I have changed the relevant parts of the code to reflect this change i.e. where-ever we did +-1 due to the off by one bug. * refactor: Noramlize the title to remove all punctuations Since the title can sometimes contain punctuations making it very difficult to get the matching answer to the question, we originally removed all such questions. This took the question count down :( and wasn't an effective way. Therefore now we keep them but as normalized, yay! I have also updated the code documentation to make the process much clearer to anyone reading the "normalizing" code section of the wiki questions generator. * refactor: Keep answers as a list & not ",".join() Wasn't fitting in character limit so shorterned it ^^ lol. Okay, getting to the point, this mentions fix error's comment of making quiz entry except the answers as a list and not as a string which could a comma joined list. The same structure was in the json resource, where multiple answers where joined with commas. This didn't allow you to use commas in answers. So I went ahead and did a bit more than requested to change the json structure and make `answers` a list. Also now all questions are in the form of the quiz entry to keep it same through out the code and var tolerance has become a valid param in QuizEntry, this is done because it was differing between questions, if not needed this would make the process to add `var_tol` as a argument to the json easier. And that's it! Co-authored-by: Objectivitix <[email protected]> Co-authored-by: Xithrius <[email protected]> Co-authored-by: ToxicKidz <[email protected]> Co-authored-by: wookie184 <[email protected]>
* | | Update code to use 'mode' variableGravatar brad90four2021-09-07-13/+46
|/ / | | | | | | | | | | Updated the code to parse user_input depending on the color code 'mode' passed to the command. Added stub code for future color codes and embeds if mode is None or wrong code.
* | Add embed fields for Hex and RGBGravatar brad90four2021-09-07-0/+10
| |
* | Fixing flake8 errors, code styleGravatar brad90four2021-09-07-12/+10
| | | | | | | | | | Still a work in progress but commenting out stub code and unused imports. List of To-Do's still applies.
* | Continue work in progressGravatar bradtimmis2021-09-06-79/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implemented the thumbnail creation from CyberCitizen0, worked on adding some features to the program. Notable Changes: -Check if user passes in hex color -Create thumbnail based on rgb_color To-Do: -Create hex color from rgb color -Create readable rgb color from user input Co-authored-by: Mohammad Rafivulla <[email protected]>
* | Merge branch 'main' of https://github.com/python-discord/sir-lancebot into ↵Gravatar bradtimmis2021-09-06-0/+1
|\| | | | | | | color-677
| * Merge branch 'main' into async-redisache-logsGravatar ChrisJL2021-09-06-101/+83
| |\
| * | Set logging level for async-rediscache to warningGravatar Numerlor2021-09-05-0/+1
| | | | | | | | | | | | Logs below warnings aren't relevant when working on the bot
* | | Remove old filesGravatar bradtimmis2021-09-05-230/+0
| | |
* | | Move to utilities folder as recommended by XithGravatar bradtimmis2021-09-05-0/+115
| | |
* | | Move to bot/exts/fun folder to update restructureGravatar bradtimmis2021-09-05-0/+115
| | |
* | | Merge branch 'main' into color-677Gravatar Xithrius2021-09-05-184/+170
|\ \ \ | | |/ | |/|
| * | Correct an import path in AOC helpersGravatar Chris Lovering2021-09-05-1/+1
| | |
| * | Add a missing __init__.py causing event cogs not to loadGravatar Chris Lovering2021-09-05-0/+0
| | |
| * | Update rogue paths to new pathsGravatar Janine vN2021-09-05-12/+12
| | |
| * | Move game and fun commands to Fun folder, fix ddgGravatar Janine vN2021-09-05-36/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This moves all the fun commands and games into the fun folder. This commit also makes changes to the duck_game. It was setting a footer during an embed init, which is no longer possible with the version of d.py we use. Additionally, an issue with editing an embed that had a local image loaded. The workaround for the time being is to update the message, not the embed.
| * | Move snakes commands into fun folderGravatar Janine vN2021-09-05-11/+11
| | |
| * | Move practical functions into utilities folderGravatar Janine vN2021-09-05-5/+5
| | | | | | | | | | | | | | | | | | Separates out the useful/practical seasonal bot features from the evergreen folder into a "utilities" folder. Adjusts the paths to resources to reflect the folder move.
| * | Move internal eval and rename utils to coreGravatar Janine vN2021-09-05-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Part of this restructure involves splitting out the massive evergreen folder into a `fun` folder and then a `utilities` folder. To help with that we've rename the `util` folder to `core`. The core functions to run the bot have been moved into this folder. `.source`, `.ping`, and `.int e` have been moved into this folder.
| * | Update paths to new resource linksGravatar Janine vN2021-09-04-5/+5
| | | | | | | | | | | | | | | | | | Additionally, this commit fixes an error with the pridepfp command. The avatar image now uses discord.py's v.20 avatar.url instead of avatar_url
| * | Move Hanukkah to Holidays folderGravatar Janine vN2021-09-04-0/+0
| | | | | | | | | | | | | | | Hannukah was previously in the Christmas folder, which was weird. This now moves it to its own folder under Holidays.
| * | Move Pride to Holidays folderGravatar Janine vN2021-09-04-4/+4
| | | | | | | | | | | | | | | Moves the Pride features to the Holidays folder. Corrected the paths to reflect this change.
| * | Move Valentines to Holidays folderGravatar Janine vN2021-09-04-8/+8
| | | | | | | | | | | | | | | Moves the valentine's day features to the holidays folders. Corrected the paths to reflect the folder moves.
| * | Move Halloween to Holidays folderGravatar Janine vN2021-09-04-7/+9
| | | | | | | | | | | | | | | Moves all the hallowen features to the holidays folder. Also updates the paths to reflect the folder moves.
| * | Move Easter to Holidays FolderGravatar Janine vN2021-09-04-10/+10
| | | | | | | | | | | | | | | | | | This moves the easter seasonal features into a more cohesive holidays/easter folder. Additionally, this splits out earth day into its own holiday folder.
| * | Move AoC and Hacktoberfest into events folderGravatar Janine vN2021-09-04-3/+3
| |/ | | | | | | | | | | | | | | | | | | Moves Advent of Code and Hacktoberfest into an events folder. Although these are roughly associated with holidays, they are standalone events that we have participated in in the past. Therefore they're being moved to an events folder separate from the "fun" or "holidays" folders.
| * Merge branch 'main' into teapot-supportGravatar ChrisJL2021-09-03-347/+361
| |\
| * | Handle status not found with 404 pictureGravatar bradtimmis2021-09-01-1/+8
| | | | | | | | | | | | | | | Hard-coded in the appropriate 404 jpgs for wehn a status code does not exist from the dog or cat urls.
| * | Remove unnecessary Embed from set_footerGravatar bradtimmis2021-09-01-1/+1
| | |
| * | Fix Embed footer text, catch 302 responseGravatar brad90four2021-08-31-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | The unknown error embed was not using the text correctly, using the suggestion from wookie to fix. The "404" response from the dog URL returns 302 instead, changed the elif to check if 302 response is returned.
| * | All ranges are inclusiveGravatar brad90four2021-08-31-2/+2
| | | | | | | | | | | | | | | Changed the range check for the status code. Previously was 599, now 600.
| * | Fix Dog instead of Cat, code error embed updateGravatar brad90four2021-08-31-14/+8
| | | | | | | | | | | | | | | | | | | | | Corrected the URL for the Dog embed (was mistakenly using CAT URL. Created a standalone ERROR_LENGTH_EMBED to send if the code used by the user is not in the range 100, 599. Changed the response status check to include 299 (to be proper).
| * | Add returns after sending embedGravatar brad90four2021-08-31-0/+2
| | | | | | | | | | | | | | | This will avoid sending the error embed even when the status codes are ok.
| * | Remove unnecessary commaGravatar brad90four2021-08-31-1/+1
| | |
| * | Add logic to handle acceptable range of codesGravatar brad90four2021-08-31-9/+22
| | | | | | | | | | | | Also updated the line breaks, black was formatting on save locally.