issue_comments

Table actions

4 rows where author_association = "CONTRIBUTOR" and issue = 574097799 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 2

issue 1

  • Documentation of DataArray does not warn that inferring dimension names is deprecated · 4 ✖

author_association 1

  • CONTRIBUTOR · 4 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
594128647 https://github.com/pydata/xarray/issues/3820#issuecomment-594128647 https://api.github.com/repos/pydata/xarray/issues/3820 MDEyOklzc3VlQ29tbWVudDU5NDEyODY0Nw== seth-p 7441788 2020-03-03T19:34:39Z 2020-03-03T19:34:39Z CONTRIBUTOR

Note that inferring dimensions from coords when it is a list of tuples does still work (with no deprecation warning): ``` In [1]: import numpy as np, xarray as xr

In [2]: xr.DataArray(np.zeros((2, 2)), coords=[('x', [1, 2]), ('y', [1, 2])])
Out[2]: <xarray.DataArray (x: 2, y: 2)> array([[0., 0.], [0., 0.]]) Coordinates: * x (x) int64 1 2 * y (y) int64 1 2 ```

 
{
    "total_count": 2,
    "+1": 2,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Documentation of DataArray does not warn that inferring dimension names is deprecated 574097799
594014351 https://github.com/pydata/xarray/issues/3820#issuecomment-594014351 https://api.github.com/repos/pydata/xarray/issues/3820 MDEyOklzc3VlQ29tbWVudDU5NDAxNDM1MQ== sjvrijn 8833517 2020-03-03T15:38:05Z 2020-03-03T15:38:05Z CONTRIBUTOR

Waiting a few more months until it will definitely not be a problem for anyone seems fair to me :+1: 

{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Documentation of DataArray does not warn that inferring dimension names is deprecated 574097799
593874263 https://github.com/pydata/xarray/issues/3820#issuecomment-593874263 https://api.github.com/repos/pydata/xarray/issues/3820 MDEyOklzc3VlQ29tbWVudDU5Mzg3NDI2Mw== sjvrijn 8833517 2020-03-03T10:20:08Z 2020-03-03T10:20:08Z CONTRIBUTOR

I think that inferring dimension-names from the coords-dict is the most intuitive way to define a DataArray.

Passing a dictionary for coords is in my opinion the clearest way to indicate which coordinates belong to which dimension, so then why do I have to specify the same dimension names again?

An example of how I create them from my current project: values = xr.DataArray( values, coords={'n_high': n_highs, 'n_low': n_lows, 'rep': repetitions, 'model': models, 'idx': range(n_test_samples),}, dims=['n_high', 'n_low', 'rep', 'model', 'idx'], <-- repeated dim names attrs=attributes, )

If you expect almost everyone to use CPython or 3.7+ anyway, then I don't actually see any drawbacks, while it would regularly make code shorter and less repetitive.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Documentation of DataArray does not warn that inferring dimension names is deprecated 574097799
593673608 https://github.com/pydata/xarray/issues/3820#issuecomment-593673608 https://api.github.com/repos/pydata/xarray/issues/3820 MDEyOklzc3VlQ29tbWVudDU5MzY3MzYwOA== sjvrijn 8833517 2020-03-02T23:18:47Z 2020-03-02T23:18:47Z CONTRIBUTOR

@max-sixty Thanks for the tip. In the end it meant just changing the last line on dims. The paragraph on the coords argument is still valid after all.

On a related note: according to #727 (PR #993), this was deprecated since key-order in dictionaries was arbitrary at the time of that issue. However, their order is fixed since Python3.7, as noted in the documentation:

Changed in version 3.7: Dictionary order is guaranteed to be insertion order. This behavior was an implementation detail of CPython from 3.6.

I guess it's still too soon to 'un-deprecate' this behavior again? 👼 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Documentation of DataArray does not warn that inferring dimension names is deprecated 574097799

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]);