issue_comments

Table actions

7 rows where issue = 1366751031 and user = 25202257 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

user 1

  • felix-hilden · 7 ✖

issue 1

  • Add sphinx-codeautolink extension to docs build · 7 ✖

author_association 1

id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1407385267 https://github.com/pydata/xarray/pull/7011#issuecomment-1407385267 https://api.github.com/repos/pydata/xarray/issues/7011 IC_kwDOAMm_X85T4v6z felix-hilden 25202257 2023-01-28T12:06:59Z 2023-01-28T12:07:46Z NONE

I'm not familiar with how :suppress: works, other than the fact that the documentation states that warnings and errors are ignored. But I see that the whole block is invisible, so maybe it is executed only internally by IPython and all output is suppressed. If this is the case, consider adding something like:

```rst .. autolink-preface::

import numpy as np
import pandas as pd
import xarray as xr
...

`` or you can also set a [global preface](https://sphinx-codeautolink.readthedocs.io/en/latest/reference.html#confval-codeautolink_global_preface) inconf.py`!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add sphinx-codeautolink extension to docs build 1366751031
1405517779 https://github.com/pydata/xarray/pull/7011#issuecomment-1405517779 https://api.github.com/repos/pydata/xarray/issues/7011 IC_kwDOAMm_X85Txn_T felix-hilden 25202257 2023-01-26T19:38:04Z 2023-01-26T19:38:04Z NONE

Wow, it's green! I still see some matching warnings though 🤔 I can have a closer look later if you care about them.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add sphinx-codeautolink extension to docs build 1366751031
1405482835 https://github.com/pydata/xarray/pull/7011#issuecomment-1405482835 https://api.github.com/repos/pydata/xarray/issues/7011 IC_kwDOAMm_X85TxfdT felix-hilden 25202257 2023-01-26T19:15:21Z 2023-01-26T19:15:21Z NONE

Yep we'll definitely follow what IPython does, now available in sphinx-codeautolink==0.13.2!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add sphinx-codeautolink extension to docs build 1366751031
1383282969 https://github.com/pydata/xarray/pull/7011#issuecomment-1383282969 https://api.github.com/repos/pydata/xarray/issues/7011 IC_kwDOAMm_X85SczkZ felix-hilden 25202257 2023-01-15T23:24:05Z 2023-01-15T23:24:05Z NONE

That actually might be something for you to change. See the issue and in particular this IPython doc on the lexer implementation. It states that blocks not starting with a console prompt should be considered ordinary (IPython) code. So In/Out won't be expected. Now maybe comments are or should be an exception, but I suspect by moving the comment elsewhere you might get it to pass!

If you refuse, I'll have a look at IPython's implementation and how they handle comments 😄 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add sphinx-codeautolink extension to docs build 1366751031
1383271546 https://github.com/pydata/xarray/pull/7011#issuecomment-1383271546 https://api.github.com/repos/pydata/xarray/issues/7011 IC_kwDOAMm_X85Scwx6 felix-hilden 25202257 2023-01-15T22:35:54Z 2023-01-15T22:35:54Z NONE

Oh yikes, that should also be fixed now! I hope the parsing logic is finally sound.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add sphinx-codeautolink extension to docs build 1366751031
1304540480 https://github.com/pydata/xarray/pull/7011#issuecomment-1304540480 https://api.github.com/repos/pydata/xarray/issues/7011 IC_kwDOAMm_X85NwbVA felix-hilden 25202257 2022-11-05T12:47:14Z 2022-11-05T12:47:14Z NONE

Yep, a weird interaction between rST processing and the IPython transformer. This class of issues fixed in sphinx-codeautolink==0.12.1!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add sphinx-codeautolink extension to docs build 1366751031
1304497912 https://github.com/pydata/xarray/pull/7011#issuecomment-1304497912 https://api.github.com/repos/pydata/xarray/issues/7011 IC_kwDOAMm_X85NwQ74 felix-hilden 25202257 2022-11-05T11:01:52Z 2022-11-05T11:01:52Z NONE

Hi, it looks like the print output is recognised as Python code and parsed. This shouldn't be happening since your what's-new correctly contains the Out [...]: prefixes which we use to ignore lines for parsing (although I couldn't find the specific case you presented). I'll test the multiline outputs and provide an update to you.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add sphinx-codeautolink extension to docs build 1366751031

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