update type-hinting for parameters
There are a lot of places where the typing is not proper, ie param: int = None instead of param: Optional[int] = None. This difference causes type checkers to complain user side when providing None directly or not providing any param to recieve the default None. I only fixed for user facing functions to be minimal.
This pull request introduces 1 alert when merging cb5824843ec4cae2553ef94092fcf7bb8511f7d7 into 8686d02963c04545c59a13f137467df28aa47236 - view on LGTM.com
new alerts:
- 1 for Unused import
This pull request introduces 3 alerts when merging 7d98a07b427bd0b4f89a90d8443f131c83910544 into 8686d02963c04545c59a13f137467df28aa47236 - view on LGTM.com
new alerts:
- 3 for Unused import
This pull request introduces 1 alert when merging 7354f58c81ef802912b137337a8c8ad788ad93a4 into 8686d02963c04545c59a13f137467df28aa47236 - view on LGTM.com
new alerts:
- 1 for Unused import
@Distortedlogic I don't know if you saw, given the noise in the status (sorry!). But test_numpy_sensor fails: https://app.circleci.com/pipelines/github/RedisAI/redisai-py/1087/workflows/cad0331e-f5b4-4ac8-94ed-205c98e71fcd/jobs/1927.
I approved the code, I thought it looked good.







