home / github

Menu
  • Search all tables
  • GraphQL API

issue_comments

Table actions
  • GraphQL API for issue_comments

6 rows where issue = 432058005 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 4

  • dazza-codes 2
  • shoyer 2
  • equaeghe 1
  • DocOtak 1

author_association 3

  • NONE 3
  • MEMBER 2
  • CONTRIBUTOR 1

issue 1

  • Optional extras to manage dependencies · 6 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
660142287 https://github.com/pydata/xarray/issues/2888#issuecomment-660142287 https://api.github.com/repos/pydata/xarray/issues/2888 MDEyOklzc3VlQ29tbWVudDY2MDE0MjI4Nw== equaeghe 601177 2020-07-17T14:36:57Z 2020-07-17T14:36:57Z NONE

Even if version information is (not yet) available, this would be very useful. Currently, when packaging xarray for downstream distributions, it is a very burdensome task to piece together which optional dependencies there are in the first place. I understand that dealing with the backlog makes this a rather tedious task, but making it a requirement going forward to add such dependency information whenever a dependency is changed or added would be required in any case.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Optional extras to manage dependencies 432058005
486024154 https://github.com/pydata/xarray/issues/2888#issuecomment-486024154 https://api.github.com/repos/pydata/xarray/issues/2888 MDEyOklzc3VlQ29tbWVudDQ4NjAyNDE1NA== DocOtak 868027 2019-04-24T00:41:49Z 2019-04-24T00:41:49Z CONTRIBUTOR

Some of my workflows involve the manual creation and destruction of virtualenvs. On occasion, I've found myself wanting a pip install xarray[complete] much in the same way dask will do. The difference between dask and xarray, however, is that the "complete" submodules are part of dask and not optional external third party dependencies.

Alternatively, it might be nice to be able to query xarray for what its current serialization capabilities are.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Optional extras to manage dependencies 432058005
485922055 https://github.com/pydata/xarray/issues/2888#issuecomment-485922055 https://api.github.com/repos/pydata/xarray/issues/2888 MDEyOklzc3VlQ29tbWVudDQ4NTkyMjA1NQ== dazza-codes 240623 2019-04-23T18:31:02Z 2019-04-23T18:31:02Z NONE

WRT points: - right, it's not tracked and that's the motivation for this issue - just because it's not widely done is not a reason to ignore it and does not justify not doing it - IMO it is not widely done because many science projects are lapse in managing version dependencies using SEM-VER best practices, leaving it to the consumers, who are often clueless about these matters, to wonder why their project crashes with one or more dependencies that they don't understand nor care to dive into at any depth - many science projects avoid the whole problem by lagging with 0.x releases, failing to commit to stable, reliable APIs with at least 1.x releases - by adopting better SEM-VER practices, across the board, it should provide more confidence to arrive at 1.x releases. But this is not my battle to wage, I'm just the messenger for thousands of consumers who have no idea what versions are compatible or not.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Optional extras to manage dependencies 432058005
485882338 https://github.com/pydata/xarray/issues/2888#issuecomment-485882338 https://api.github.com/repos/pydata/xarray/issues/2888 MDEyOklzc3VlQ29tbWVudDQ4NTg4MjMzOA== shoyer 1217238 2019-04-23T16:41:59Z 2019-04-23T16:41:59Z MEMBER

Given that our optional dependencies can all be added individually, I'm a little skeptical that options like pip install xarray[netcdf4] provide much value over pip install xarray netcdf4. But I do appreciate the virtue of making this information programmatically accessible, and certainly would have no objection to contributors here.

But do note: - We currently do not keep track of version requirements for optional dependencies. It would take some investigation/testing to figure out what works. - This is not a widespread practice among Python data/numerical computing projects. For example, pandas does not do this.

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Optional extras to manage dependencies 432058005
485868641 https://github.com/pydata/xarray/issues/2888#issuecomment-485868641 https://api.github.com/repos/pydata/xarray/issues/2888 MDEyOklzc3VlQ29tbWVudDQ4NTg2ODY0MQ== dazza-codes 240623 2019-04-23T16:04:41Z 2019-04-23T16:18:40Z NONE

Since xarray is operating as a meta-package with various wrappers, it is the only place that can apply version compatibility on it's dependencies. If there are optional dependencies, the python package extras are the process to manage them (not a README). The SEM-VER standards allow specification of more than one version for any dependency (so long as it complies with SEM-VER standards).

Essentially, this is a request for more sophistication in the setup.py file - https://github.com/pydata/xarray/blob/master/setup.py

Consider, for example, how Apache Airflow does it: - https://github.com/apache/airflow/blob/master/setup.py - https://github.com/apache/airflow/blob/master/setup.py#L353

{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Optional extras to manage dependencies 432058005
482434962 https://github.com/pydata/xarray/issues/2888#issuecomment-482434962 https://api.github.com/repos/pydata/xarray/issues/2888 MDEyOklzc3VlQ29tbWVudDQ4MjQzNDk2Mg== shoyer 1217238 2019-04-12T04:42:41Z 2019-04-12T04:42:41Z MEMBER

We test xarray continuously against the latest released versions of all these dependencies (and older versions, too, in many cases). For xarray and netCDF4, for example, you could simply use pip install xarray netcdf4.

Are there particular subpackages you would suggest?

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
  Optional extras to manage dependencies 432058005

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 17.177ms · About: xarray-datasette