Skip to content

Conversation

@R5dan
Copy link
Contributor

@R5dan R5dan commented Jan 30, 2025

These are some of my ideas for the screener class.
I thought it would be easier to put them in a PR then to try to explain them
Will remove any as wanted.

Changes:

  • Better type hints
  • Moves is-in to classmethod
  • Doesn't import all of screener - have to go from yfinance.screener import ...
  • Added Query class to control entire query
  • Added QueryHead class for easy access to region exchange sector
  • Added screen to queries - allows screening from there

@ValueRaider
Copy link
Collaborator

Can you respond to this #2175 (comment)

@R5dan
Copy link
Contributor Author

R5dan commented Jan 30, 2025

Sorry just seen, I will fix and make a new PR.

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.

2 participants