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

Update live.py #86

Closed
wants to merge 4 commits into from
Closed

Update live.py #86

wants to merge 4 commits into from

Conversation

k-prabin
Copy link

@k-prabin k-prabin commented Nov 11, 2024

fix

@divyankm
Copy link

Pls update this line as well.

  1. history.py#L41
  2. archives.py#L150

@skyler-guha
Copy link

skyler-guha commented Nov 18, 2024

This seems to work. Thanks!!
(only tested it on historical data tho)

Edit:
does not work on when getting index df.
I think this lib does not get maintained anymore maybe.
Last commit was on Dec 24, 2023.
I might be willing to contribute to try to fix the issues, but no clue if the maintainer still takes PRs.
I will keep an eye on this.

Incase anyone wants to install the forked repo with the patches:
pip install https://github.com/k-prabin/jugaad-data/archive/refs/heads/patch-1.zip

@k-prabin
Copy link
Author

Pls update this line as well.

  1. history.py#L41
  2. archives.py#L150

done

@k-prabin
Copy link
Author

working?

@buzzvolt
Copy link

please check use of brotli when dealing with br responses and see if requests could use that -- I have pr#88

@buzzvolt
Copy link

PR#88 could help resolve move to HTTP 2 and also friendly with cloud instances

@buzzvolt
Copy link

guys try out #88 if that solves your problem

@sevakram
Copy link
Contributor

Thanks for the contribution, Sorry, I saw this late, I am combining this with other changes in my PR.

@sevakram sevakram closed this Dec 21, 2024
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.

5 participants