Market Status

[R5dan]

Adds a class for Market Status. Different from Market Summary although that does include part of the data.

[R5dan]

Ruff is wrong by the way. The imports are used

[ValueRaider]

Ruff is wrong by the way. The imports are used

Ruff is right. You're looking at the wrong file.

[R5dan]

Ruff is wrong by the way. The imports are used

Ruff is right. You're looking at the wrong file.

Yeah, hadn't read it properly. Don't know why I was even in search.py

[ValueRaider]

Bug:

Traceback (most recent call last):
  File "./test.py", line 1, in <module>
    import yfinance as yf
  File "yfinance/__init__.py", line 23, in <module>
    from .status import Status
  File "yfinance/status.py", line 7, in <module>
    from datetime import (
ImportError: cannot import name 'deltatime' from 'datetime' (/usr/lib64/python3.12/datetime.py). Did you mean: 'datetime'?

[R5dan]

Bug:

Traceback (most recent call last):
  File "./test.py", line 1, in <module>
    import yfinance as yf
  File "yfinance/__init__.py", line 23, in <module>
    from .status import Status
  File "yfinance/status.py", line 7, in <module>
    from datetime import (
ImportError: cannot import name 'deltatime' from 'datetime' (/usr/lib64/python3.12/datetime.py). Did you mean: 'datetime'?

Fixed that, sorry its timedelta not deltatime

[R5dan]

I currently am getting:

requests.exceptions.SSLError: HTTPSConnectionPool(host='query1.finance.yahoo.com', port=443): Max retries exceeded with url: /v1/test/getcrumb (Caused by SSLError(SSLCertVerificationError(1, "[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: Hostname mismatch, certificate is not valid for 'query1.finance.yahoo.com'. (_ssl.c:1020)")))

when requesting data from yfinance. How do I stop this?

[ValueRaider]

when requesting data from yfinance. How do I stop this?

wrong thread

[R5dan]

when requesting data from yfinance. How do I stop this?

wrong thread

? I am only getting this with this patch. That is why it is here not on a separate issue.

[ValueRaider]

Max retries exceeded with url: /v1/test/getcrumb

Nothing to do with this branch, and doesn't happen to me.

Also your branch is riddled with bugs.

[R5dan]

@ValueRaider fixed bugs.

[ValueRaider]

yf.Status("US")

  File "./test.py", line 7, in <module>
    yf.Status("US")
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "yfinance/status.py", line 26, in __init__
    self._parse_data(True)
  File "yfinance/status.py", line 55, in _parse_data
    self._fetch_data()
  File "yfinance/status.py", line 45, in _fetch_data
    type_change = self._raw_data["duration"][0]
                  ~~~~~~~~~~~~~~^^^^^^^^^^^^
KeyError: 'duration'

[R5dan]

  File "./test.py", line 7, in <module>
    x = yf.Status("US", session=session)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "yfinance/status.py", line 26, in __init__
    self._parse_data(True)
  File "yfinance/status.py", line 55, in _parse_data
    self._fetch_data()
  File "yfinance/status.py", line 45, in _fetch_data
    type_change = self._raw_data["duration"][0]

What code did you use for this? And what is the actual error @ValueRaider

[ValueRaider]

commit fd309bd

[R5dan]

I can't tell if you are telling me to commit it or that that is the error? @ValueRaider

[ValueRaider]

I thought you were asking code version, because the code I ran is in the stack trace. I've added error message.

[R5dan]

I think they must have edited that API. That used to be how long that section would last (when yfit_market_status changed). Also just noticed spelling mistake in the commit message

[R5dan]

Is there anything else you want me to add or change? @ValueRaider

[ValueRaider]

Like with your other branch, don't unpack the dict into class attributes, just return a dict. You can parse its values but still return a dict.

[R5dan]

Done @ValueRaider anything else?

[R5dan]

@ValueRaider Can this be merged?

[ValueRaider]

Almost, but where you parse into attributes e.g. fromisoformat just overwrite the dict. Seems less work for everyone. Basically reduce to a simple function with optional argument raw=True.

[R5dan]

What about storing it? It seems silly to reparse data every time when I feel more people will use the parsed on (especially as is_open is a custom property I defined as that is it's most likely use. The other 4 are also all going to be common uses. I'm happy to change it but can you tell me what to change as I don't get what your asking for. @ValueRaider

[R5dan]

Closing as added to market summary