You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "AlenkaF (via GitHub)" <gi...@apache.org> on 2024/03/04 11:57:44 UTC
[I] [Python][CI] Failing test_dateutil_tzinfo_to_string due to new release of python-dateutil [arrow]
AlenkaF opened a new issue, #40337:
URL: https://github.com/apache/arrow/issues/40337
### Describe the bug, including details regarding any error messages, version, and platform.
`test_dateutil_tzinfo_to_string` started failing with:
tz = dateutil.tz.gettz('Europe/Paris')
> assert pa.lib.tzinfo_to_string(tz) == 'Europe/Paris'
E AssertionError: assert 'Europe/Monaco' == 'Europe/Paris'
E - Europe/Paris
E + Europe/Monaco
see: https://github.com/ursacomputing/crossbow/actions/runs/8103088008/job/22164107523#step:6:4305.
This is most probably due to new release of `python-dateutil` package.
The test is failing on the CI now also, with dateutil `2.9.0`: https://ci.appveyor.com/project/ApacheSoftwareFoundation/arrow/builds/49322144?fullLog=true
### Component(s)
Continuous Integration, Python
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Python][CI] Failing test_dateutil_tzinfo_to_string due to new release of python-dateutil [arrow]
Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #40337:
URL: https://github.com/apache/arrow/issues/40337#issuecomment-1991468061
A temporary skip was added in https://github.com/apache/arrow/pull/40486
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Python][CI] Failing test_dateutil_tzinfo_to_string due to new release of python-dateutil [arrow]
Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #40337:
URL: https://github.com/apache/arrow/issues/40337#issuecomment-1976516507
I can't reproduce this on Linux, so most probably a Windows issue (or related to the exact tzdata being used there, since for me locally it's using the system provided data)
Might be good to report upstream?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Python][CI] Failing test_dateutil_tzinfo_to_string due to new release of python-dateutil [arrow]
Posted by "AlenkaF (via GitHub)" <gi...@apache.org>.
AlenkaF commented on issue #40337:
URL: https://github.com/apache/arrow/issues/40337#issuecomment-1976537652
Agree with you Joris. I would first like to check the behaviour of dateutil, as you mention, but also can't do that on my M1. I could in any case open an issue upstream before, if we think that makes sense.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Python][CI] Failing test_dateutil_tzinfo_to_string due to new release of python-dateutil [arrow]
Posted by "jorisvandenbossche (via GitHub)" <gi...@apache.org>.
jorisvandenbossche commented on issue #40337:
URL: https://github.com/apache/arrow/issues/40337#issuecomment-1976525528
The logic we use to convert a dateutil tz to a string:
https://github.com/apache/arrow/blob/2b194ad222f4dc8ecf2eb73539ab8cab5b1fc5e7/python/pyarrow/src/arrow/python/datetime.cc#L554-L568
(so it might also be that something changed under the hood in dateutil that messes that u. It would be good to verify if the actual `tz` object in the snippet above, before conversion to a string by pyarrow, also shows "Europe/Monaco")
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Python][CI] Failing test_dateutil_tzinfo_to_string due to new release of python-dateutil [arrow]
Posted by "AlenkaF (via GitHub)" <gi...@apache.org>.
AlenkaF commented on issue #40337:
URL: https://github.com/apache/arrow/issues/40337#issuecomment-1976418621
cc @raulcd
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [Python][CI] Failing test_dateutil_tzinfo_to_string due to new release of python-dateutil [arrow]
Posted by "raulcd (via GitHub)" <gi...@apache.org>.
raulcd commented on issue #40337:
URL: https://github.com/apache/arrow/issues/40337#issuecomment-1976777901
The same issue is being reproduced on my PR on the conda feedstock:
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=888527&view=logs&j=ab98fee7-5bc6-5c2f-a410-3ab9b2f2e8ca&t=472408ae-fc68-5791-981c-69ea41d2d692&l=38399
```
def test_dateutil_tzinfo_to_string():
pytest.importorskip("dateutil")
import dateutil.tz
tz = dateutil.tz.UTC
assert pa.lib.tzinfo_to_string(tz) == 'UTC'
tz = dateutil.tz.gettz('Europe/Paris')
> assert pa.lib.tzinfo_to_string(tz) == 'Europe/Paris'
E AssertionError: assert 'Europe/Monaco' == 'Europe/Paris'
E - Europe/Paris
E + Europe/Monaco
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org