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

Multithreading, terminal colors, improved prompts, and automatic module installation #54

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Music-Dude
Copy link

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
I added multithreading, terminal colors, improved prompts, and added automatic module installing. I also cleaned up the code for consistency.

Describe the changes proposed in the pull request A quick description of the changes made to the repository
The program can generate codes more efficiently, has colored messages in the terminal, asks the user until they provide a valid input, and installs modules if the user wants.

What is the current behavior? (You can also link to an open issue here)
The program only runs on one thread. Everything is black and white. The program raises an error if an invalid value is given by the user. The program only tells the user a command to run in order to install dependencies.

What is the new behavior (if this is a feature change)?
The program can now run multiple threads at once for faster generation. The prompts are now in colors to be more aesthetically pleasing. The prompts now ask until you provide a valid answer instead of throwing errors. If the modules are not installed the program lets the user choose if they would like to install them.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No. It only adds an additional prompt to the start.

Other information:
I removed unused functions, changed all quotes to single quotes for consistency and separated introduction into separate functions.

@ykonyshev
Copy link

@Music-Dude Code looks pretty good, but it won't work as intended, because Discord API limits the number of requests sent by a single user.

@Vicky-Boi
Copy link

how do i use this pull request in this program? im a big github noob, so i dont know how to actually do it. pls someone help!

@TotallyNotInUse
Copy link

@Music-Dude Code looks pretty good, but it won't work as intended, because Discord API limits the number of requests sent by a single user.

This could be fixed if a queue system was added (although speed would be much slower)

@Aholicknight
Copy link
Contributor

how do i use this pull request in this program? im a big github noob, so i dont know how to actually do it. pls someone help!

@Vicky-Boi Use the fork from the pull request, @Music-Dude has a fork of this so you can download it from his and use it.

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.

None yet

5 participants