search

califyn / time-cord

A Python library for Discord time management tools
MIT License
0 stars 5 forks source link

Improvements #11

Closed LicketySpliket closed 3 years ago

LicketySpliket commented 3 years ago

I added a class structure to the monitor.py file. This allows you to easily toggle debug=True vs debug=False to control whether you want to print output. Furthermore, I added code that suppresses insecure SSL errors, which can be raised a lot (especially on Andover wifi).

califyn commented 3 years ago

Wait, why is requests/SSL/whatever even necessary? I'm just asking because I thought there would be no components that require an internet connection, but I might be wrong (in particular I know very little about how the OCR tool works...)

LicketySpliket commented 3 years ago

I believe the OCR tool has to download a training dataset, and to do so it downloads it over an SSL encrypted connection. Idk why, but andover WiFi has a problem with SSL sometimes. I turned off certificate verification because the dataset doesn’t really need To be encrypted as it’s not card info or personal info.

Sent from my iPhone

On Oct 21, 2020, at 12:21 AM, Ali Cy notifications@github.com wrote:

 Wait, why is requests/SSL/whatever even necessary? I'm just asking because I thought there would be no components that require an internet connection, but I might be wrong (in particular I know very little about how the OCR tool works...)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

califyn commented 3 years ago

Okay, thanks so much for the improvements!

(Also, note to self in the future: I think the thing it's loading might be the model weights, actually. I thought they were downloaded onto the computer during installation but I guess not. Maybe I'll look for some alternatives later on?)