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

[Bug] stocks/ba/popular - Function breaks the terminal #3784

Closed
deeleeramone opened this issue Dec 15, 2022 · 6 comments
Closed

[Bug] stocks/ba/popular - Function breaks the terminal #3784

deeleeramone opened this issue Dec 15, 2022 · 6 comments
Assignees
Labels
bug Fix bug

Comments

@deeleeramone
Copy link
Contributor

/stocks/ba/popular

The terminal has to be forced out with a keyboard interrupt.

(🦋) /stocks/ba/ $ popular

/Users/danglewood/GitHub/GST1/miniconda3/envs/obbx86/lib/python3.10/site-packages/psaw/PushshiftAPI.py:192: UserWarning: Got non 200 code 404
  warnings.warn("Got non 200 code %s" % response.status_code)
/Users/danglewood/GitHub/GST1/miniconda3/envs/obbx86/lib/python3.10/site-packages/psaw/PushshiftAPI.py:180: UserWarning: Unable to connect to pushshift.io. Retrying after backoff.
  warnings.warn("Unable to connect to pushshift.io. Retrying after backoff.")
@deeleeramone deeleeramone added the bug Fix bug label Dec 15, 2022
@jmaslek
Copy link
Collaborator

jmaslek commented Dec 15, 2022

Looks like pushshift is broken

Screenshot 2022-12-15 at 11 36 13 AM

@jmaslek
Copy link
Collaborator

jmaslek commented Jan 18, 2023

Looks like a month later pushshift is still down. Will likely need to deprecate this function until someone adds a different reddit api.

@montezdesousa montezdesousa self-assigned this Jan 18, 2023
@montezdesousa
Copy link
Contributor

I was putting a timeout here #3999, do you prefer to deprecate?

@jmaslek
Copy link
Collaborator

jmaslek commented Jan 18, 2023

Have you gotten this specific function to work or does it just timeout?

@montezdesousa
Copy link
Contributor

Have you gotten this specific function to work or does it just timeout?

Just timeout, but looking at this dmarx/psaw#107 it might really need to use a different reddit api

@montezdesousa
Copy link
Contributor

Fix here #3999

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fix bug
Projects
None yet
Development

No branches or pull requests

3 participants