issue_comments

Table actions

9 rows where author_association = "NONE" and user = 25202257 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

user 1

  • felix-hilden · 9 ✖

author_association 1

  • NONE · 9 ✖
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
1247260062 https://github.com/pydata/xarray/issues/7010#issuecomment-1247260062 https://api.github.com/repos/pydata/xarray/issues/7010 IC_kwDOAMm_X85KV62e felix-hilden 25202257 2022-09-14T20:20:08Z 2022-09-14T20:20:08Z NONE

Yeah I was just messing around 😄 Being conservative with random packages is advised. Buuut, 0.12 released just now with the extra links! Lemme know how it goes 👍

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Use sphinx-codeautolink in docs? 1366657155
1242995961 https://github.com/pydata/xarray/issues/7010#issuecomment-1242995961 https://api.github.com/repos/pydata/xarray/issues/7010 IC_kwDOAMm_X85KFpz5 felix-hilden 25202257 2022-09-11T16:12:12Z 2022-09-11T16:12:53Z NONE

Hi, I've indeed implemented links for assignment targets, bare names and annotated function arguments recently. The changes are still unreleased though, because I want to include match support in the same batch (felix-hilden/sphinx-codeautolink#110). Feel free to try "master" meanwhile, but I'll let you know when it's released 👍 (to stop this stalking and/or harassment, press 🚀)

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Use sphinx-codeautolink in docs? 1366657155

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