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

Xrange to Range | Added Session and Timeout to requests | unicode was… #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shauryauppal
Copy link
Contributor

@shauryauppal shauryauppal commented Oct 3, 2020

  1. Xrange to Range for Python3.5+ compatible code
  2. Added Session and timeout for smooth file downloads from rbi
  3. unicode was type line was creating an issue so modified

@shauryauppal
Copy link
Contributor Author

@knadh

@knadh
Copy link
Collaborator

knadh commented Mar 8, 2021

Sorry, missed this. I vaguely remember RBI's website changing, breaking the script. Have you been using the lib? If yes, can you confirm that the lib works as is?

@shauryauppal
Copy link
Contributor Author

shauryauppal commented May 19, 2021 via email

@shauryauppal
Copy link
Contributor Author

Last, when I did the commit everything was working perfectly with the changes, don't know about the current state.

Some much-needed changes were like Xrange doesn't work anymore on Python3+ versions. So those changes to make code python3 compatible were also done.

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

2 participants