ValueRaider comments

Results 898 comments of


                                            ValueRaider

Reduce `info[]` use

> `yf.basic_market_data` Good suggestion, and probably the best option. Only challenge is ensuring most/all affected users know & switch.

Reduce `info[]` use

Do you mind trying branch `feature/prune-info`? Hasn't implemented `basic_market_data`, just tells users to use other methods.

Reduce `info[]` use

@cmjordan42 I've now removed the lazy-loading stuff. I only thought it would help in cases like `i = dat.info ; i["marketCap"]` - they might leave the first line. But your...

Reduce `info[]` use

> implement that `basic_market_data` call Done > worthwhile to get out there sooner I want feedback first that the messages make transition easy.

Reduce `info[]` use

Branch `feature/prune-info`

Reduce `info[]` use

@gene-git I fast-tracked it today, it's in latest PIP release.

Reduce `info[]` use

@gene-git wait, there's a bug pending #1340

get_earnings raises type error

@lucas03 Maybe submit a pull request? @suhaskashyaps You're issue is different, you should submit a new Issue. But first double-check bug because I can't reproduce.

get_earnings raises type error

You're thinking right area. But I'd question why it's trying to add currency in first place - inspect GitHub blame, who added it and why.

Request: dynamically extract decryption key(s) from HTML

@Meborl That's useful information, more redundancy is good.