home / github

Menu
  • GraphQL API

issue_comments

Table actions
  • GraphQL API for issue_comments

9 rows where issue = 1124237013

✎ View and edit SQL

This data as json, CSV (advanced)

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

id ▼ html_url issue_url node_id user created_at updated_at author_association body reactions issue performed_via_github_app
1030454114 https://github.com/simonw/sqlite-utils/issues/398#issuecomment-1030454114 https://api.github.com/repos/simonw/sqlite-utils/issues/398 IC_kwDOCGYnMM49a3ti simonw 9599 2022-02-05T00:14:47Z 2022-02-05T00:14:47Z OWNER I like these designs a lot. I would suggest `sqlite-utils create database.db --init-spatialite` there for consistency with the `sqlite-utils init-spatialite database.db` command. The other part of this story is how we support actually inserting spatial data from the command-line. I opened an issue about the challenges in doing that for the Python API here - #399 - but we need a good answer for the CLI too. I don't yet have any good ideas here. The `conversions=` option in the Python library was designed to cover these kinds of cases but it's pretty clunky and I don't think it's very widely used: https://sqlite-utils.datasette.io/en/stable/python-api.html#converting-column-values-using-sql-functions {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} Add SpatiaLite helpers to CLI 1124237013  
1030456717 https://github.com/simonw/sqlite-utils/issues/398#issuecomment-1030456717 https://api.github.com/repos/simonw/sqlite-utils/issues/398 IC_kwDOCGYnMM49a4WN simonw 9599 2022-02-05T00:16:42Z 2022-02-05T00:16:42Z OWNER > The one thing worth highlighting in docs is that geometry columns can only be added to existing tables. Trying to add a geometry column to a table that doesn't exist yet might mean you have a schema like `{"rowid": int, "geometry": bytes}`. Might be worth nudging people to explicitly create a table first, then add geometry columns. That's a good call. I'm happy for `sqlite-utils add-geometry-column` to throw an error if the table doesn't exist yet. {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} Add SpatiaLite helpers to CLI 1124237013  
1030521533 https://github.com/simonw/sqlite-utils/issues/398#issuecomment-1030521533 https://api.github.com/repos/simonw/sqlite-utils/issues/398 IC_kwDOCGYnMM49bIK9 simonw 9599 2022-02-05T04:25:49Z 2022-02-05T04:25:49Z OWNER For ingesting geometry data from the command-line maybe GeoJSON would be the best route? {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} Add SpatiaLite helpers to CLI 1124237013  
1030534868 https://github.com/simonw/sqlite-utils/issues/398#issuecomment-1030534868 https://api.github.com/repos/simonw/sqlite-utils/issues/398 IC_kwDOCGYnMM49bLbU simonw 9599 2022-02-05T06:03:38Z 2022-02-05T06:03:38Z OWNER @eyeseast how do you usually insert geometries at the moment? {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} Add SpatiaLite helpers to CLI 1124237013  
1030629879 https://github.com/simonw/sqlite-utils/issues/398#issuecomment-1030629879 https://api.github.com/repos/simonw/sqlite-utils/issues/398 IC_kwDOCGYnMM49bin3 eyeseast 25778 2022-02-05T13:57:33Z 2022-02-05T19:49:38Z CONTRIBUTOR I'm mostly using [geojson-to-sqlite](https://github.com/simonw/geojson-to-sqlite) at the moment. Even with shapefiles, I'm usually converting to GeoJSON and projecting to EPSG:4326 (with [ogr2ogr](https://gdal.org/programs/ogr2ogr.html)) first. I think an open question here is how much you want to leave to external libraries and how much you want here. My thinking has been that adding Spatialite helpers here would make external stuff easier, but it would be nice to have some standard way to insert geometries. I'm in the middle of adding GeoJSON and Spatialite support to [geocode-sqlite](https://github.com/eyeseast/geocode-sqlite), and that will probably use WKT. Since that's all points, I think I can just make the string inline. But for polygons, I'd generally use Shapely, which probably isn't a dependency you want to add to sqlite-utils. I've also been trying to get some of the approaches [here](https://www.gaia-gis.it/fossil/libspatialite/wiki?name=Supporting+GeoJSON) to work, but haven't had any success so far. {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} Add SpatiaLite helpers to CLI 1124237013  
1030732093 https://github.com/simonw/sqlite-utils/issues/398#issuecomment-1030732093 https://api.github.com/repos/simonw/sqlite-utils/issues/398 IC_kwDOCGYnMM49b7k9 simonw 9599 2022-02-06T01:41:37Z 2022-02-06T01:41:37Z OWNER Yeah I'd like to avoid adding any geo-dependencies to `sqlite-utils` if I can avoid it. I'm fine using stuff that's going to be available in SpatiaLite itself (provided it's available as a SQLite module) since then I don't need to add any extra Python dependencies. {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} Add SpatiaLite helpers to CLI 1124237013  
1030732222 https://github.com/simonw/sqlite-utils/issues/398#issuecomment-1030732222 https://api.github.com/repos/simonw/sqlite-utils/issues/398 IC_kwDOCGYnMM49b7m- simonw 9599 2022-02-06T01:42:19Z 2022-02-06T01:42:28Z OWNER Adding some thoughts to: - #399 {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} Add SpatiaLite helpers to CLI 1124237013  
1038336591 https://github.com/simonw/sqlite-utils/issues/398#issuecomment-1038336591 https://api.github.com/repos/simonw/sqlite-utils/issues/398 IC_kwDOCGYnMM4948JP eyeseast 25778 2022-02-13T18:48:21Z 2022-02-13T18:49:49Z CONTRIBUTOR Been chipping away at this between other things and realized `sqlite-utils init-spatialite` is probably unnecessary. Any of the other commands requires running `db.init_spatialite` to have the extension functions available, and that will do everything `init-spatialite` would do. I think it's probably worth keeping a SpatiaLite flag on `create-database` in case you wanted to create all the spatial metadata up front. Otherwise, it's going to get added the first time you run `add-geometry-column` or `create-spatial-index`, which is probably fine in most cases. {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} Add SpatiaLite helpers to CLI 1124237013  
1040965672 https://github.com/simonw/sqlite-utils/issues/398#issuecomment-1040965672 https://api.github.com/repos/simonw/sqlite-utils/issues/398 IC_kwDOCGYnMM4-C-Ao simonw 9599 2022-02-16T01:02:29Z 2022-02-16T01:02:29Z OWNER Documentation: - https://sqlite-utils.datasette.io/en/latest/cli-reference.html#create-database - https://sqlite-utils.datasette.io/en/latest/cli-reference.html#add-geometry-column - https://sqlite-utils.datasette.io/en/latest/cli-reference.html#create-spatial-index - https://sqlite-utils.datasette.io/en/latest/cli.html#spatialite-helpers {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} Add SpatiaLite helpers to CLI 1124237013  

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,
   [issue] INTEGER REFERENCES [issues]([id])
, [performed_via_github_app] TEXT);
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.738ms · About: simonw/datasette-graphql