home / github

Menu
  • GraphQL API
  • Search all tables

issue_comments

Table actions
  • GraphQL API for issue_comments

10 rows where issue = 1108564253 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

Suggested facets: reactions, created_at (date), updated_at (date)

user 8

  • jhamman 2
  • ksunden 2
  • shoyer 1
  • brews 1
  • jrbourbeau 1
  • andersy005 1
  • Illviljan 1
  • keewis 1

author_association 2

  • MEMBER 6
  • CONTRIBUTOR 4

issue 1

  • Xarray versioning to switch to CalVer · 10 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1057657161 https://github.com/pydata/xarray/issues/6176#issuecomment-1057657161 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X84_CpFJ shoyer 1217238 2022-03-03T04:32:10Z 2022-03-03T04:32:10Z MEMBER

Breaking changes will continue to be very rare, and whenever possible will be preceeded by deprecation or future warnings for multiple months.

{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253
1057653294 https://github.com/pydata/xarray/issues/6176#issuecomment-1057653294 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X84_CoIu brews 2049051 2022-03-03T04:24:14Z 2022-03-03T04:24:14Z CONTRIBUTOR

Using semver can be handy to signal breaking changes. Any thoughts on how xarray will handle breaking changes with calver? Any release can break, or during select periods, after a timeout, etc.?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253
1025382413 https://github.com/pydata/xarray/issues/6176#issuecomment-1025382413 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X849HhgN jhamman 2443309 2022-01-31T05:02:19Z 2022-01-31T05:02:19Z MEMBER

After thinking on this one a bit, I'm back to thinking we should zero-pad the months. I don't think there is a clear right choice here, with both options offering pros/cons. Thanks @ksunden in particular for sharing your perspective). I suggest we try this in 2022.02.0. See #6214 for more details.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253
1020681788 https://github.com/pydata/xarray/issues/6176#issuecomment-1020681788 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X8481l48 jrbourbeau 11656932 2022-01-25T00:19:49Z 2022-01-25T01:27:00Z CONTRIBUTOR

@jrbourbeau is this something dask has thought about?

Thanks for the ping @Illviljan. Zero-padding dates did come up in the Dask calver discussion starting here https://github.com/dask/community/issues/100#issuecomment-704445214. In a nutshell, there was a slight preference towards using zero-padding (i.e. 2022.01.0 instead of 2022.1.0) because the calendar nature of the version is more explicit and string sorting and full-fledged package sorting (like one would do with packaging.version) give the same result.

As pointed out https://github.com/dask/community/issues/100#issuecomment-704468187 either convention is valid from a Python packaging perspective. FWIW I'm not aware of any issues that have come up from Dask using a zero-padded version number. The main thing that comes to mind is when checking out git tags for a specific release (e.g. git checkout 2021.04.0 and git checkout 2021.4.0 are not equivalent). That said, to my knowledge, this hasn't been an issue in practice.

EDIT: To be clear, I'm not advocating for one convention over the other -- just providing context around Dask's decision

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253
1020669305 https://github.com/pydata/xarray/issues/6176#issuecomment-1020669305 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X8481i15 ksunden 2501846 2022-01-24T23:57:13Z 2022-01-24T23:57:13Z CONTRIBUTOR

setup.py sdist bdist_wheel creates files in dist/ that have the leading zeros removed.

I did git tag v2022.01.0 and then built the wheel/sdist and got: xarray-2022.1.0-py3-none-any.whl and xarray-2022.1.0.tar.gz and xarray.__version__ gave 2022.1.0

I do understand the appeal of leading zeros, I started off using them myself, but pypi/setuptools/etc will bring in inconsistencies that you will either need to fight against or give into the consistent way, which is sans leading zero (that is the path I chose)

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253
1020614631 https://github.com/pydata/xarray/issues/6176#issuecomment-1020614631 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X8481Vfn keewis 14808389 2022-01-24T22:28:27Z 2022-01-24T22:46:02Z MEMBER

note that PEP440 itself is not really consistent: it mentions both normal and zero-padded CalVer.

Also, in the issue in which dask discussed this, one of the people involved with the PyPA stated that the difference between omitting or keeping the zero-padding is purely aesthetic (both can be parsed using packaging.version.Version / packaging.version.parse). I assume that means we're free to choose? The YYYY.MM.X scheme seems to be fairly common now, though.

Edit: I would vote for zero-padding of the months because that will make it actually recognizable as a date.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253
1020525378 https://github.com/pydata/xarray/issues/6176#issuecomment-1020525378 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X8480_tC Illviljan 14371165 2022-01-24T20:37:32Z 2022-01-24T20:37:32Z MEMBER

Note that PEP 440 normalizes integers in version strings, so leading zeros are ignored and the version as it appears in PyPI would be 2022.1.0, as that displays the normalized version string.

On my own packages which I use calver for I have opted to not have leading zeros such that the "canonical" version matches the "normalized" version to avoid any confusion that may cause.

No opinion either way here but 2022.01.0 is an easier string to sort.

@jrbourbeau is this something dask has thought about?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253
1020519502 https://github.com/pydata/xarray/issues/6176#issuecomment-1020519502 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X8480-RO andersy005 13301940 2022-01-24T20:30:05Z 2022-01-24T20:30:05Z MEMBER

Do others have thoughts here? I would support stripping the leading zeros from the MM part of the version string in favor of consistency here.

I'm in favor of a non-zero-padded version for the benefit of having canonical/normalized versions that also match git tags/history

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253
1020463316 https://github.com/pydata/xarray/issues/6176#issuecomment-1020463316 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X8480wjU jhamman 2443309 2022-01-24T19:27:38Z 2022-01-24T19:27:38Z MEMBER

Interesting insights @ksunden, thanks for sharing!

Do others have thoughts here? I would support stripping the leading zeros from the MM part of the version string in favor of consistency here.

{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253
1020405910 https://github.com/pydata/xarray/issues/6176#issuecomment-1020405910 https://api.github.com/repos/pydata/xarray/issues/6176 IC_kwDOAMm_X8480iiW ksunden 2501846 2022-01-24T18:23:26Z 2022-01-24T18:23:26Z CONTRIBUTOR

Note that PEP 440 normalizes integers in version strings, so leading zeros are ignored and the version as it appears in PyPI would be 2022.1.0, as that displays the normalized version string.

On my own packages which I use calver for I have opted to not have leading zeros such that the "canonical" version matches the "normalized" version to avoid any confusion that may cause.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Xarray versioning to switch to CalVer 1108564253

Advanced export

JSON shape: default, array, newline-delimited, object

CSV options:

CREATE TABLE [issue_comments] (
   [html_url] TEXT,
   [issue_url] TEXT,
   [id] INTEGER PRIMARY KEY,
   [node_id] TEXT,
   [user] INTEGER REFERENCES [users]([id]),
   [created_at] TEXT,
   [updated_at] TEXT,
   [author_association] TEXT,
   [body] TEXT,
   [reactions] TEXT,
   [performed_via_github_app] TEXT,
   [issue] INTEGER REFERENCES [issues]([id])
);
CREATE INDEX [idx_issue_comments_issue]
    ON [issue_comments] ([issue]);
CREATE INDEX [idx_issue_comments_user]
    ON [issue_comments] ([user]);
Powered by Datasette · Queries took 14.21ms · About: xarray-datasette