-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add Chicago Data Sets #235
Comments
Divvy and Crimes in particular are problematic. The server needs to render the entire data set before it can start sending it to us. I put an email into Jon over at the city's data office about work arounds for this. |
Blocked until we start addressing #377 |
vforgione
pushed a commit
that referenced
this issue
Aug 21, 2018
The original signature didn't account for the function being called when the data set was still empty. This corrects that. Updates #235
vforgione
pushed a commit
that referenced
this issue
Aug 21, 2018
TIL Postgres has a maximum table name length of 63 characters. I set an upper bound of 58 on the name length of the MetaChangeset. Why 58 and not 63? Because when we create the materialized view for the data set we tack on "_view" to the table name. Updates #235
vforgione
pushed a commit
that referenced
this issue
Nov 14, 2018
The internal API was getting really nasty -- we had a bunch of one off functions that clashed in arity (positional arguments, matches, guards, options ...). The web application was also a disaster -- originally I thought it would make it easier to keep the web, admin and API separate in subapps, but that ended up making things that much more difficult. Then that leaves the elephant in the room: Socrata. We've always relied on them and all of their awful decisions. The changes here in remove some of the terrible things about Socrata integration and makes ingesting their data sets a little cleaner. Breaking Changes: - total revision of the migrations - entirely removed the `UserAdminMessage` schema - entirely removes all the outstanding ETL job stuff - entirely removes charts -- that was a really stupid idea - entirely removes exports -- again just a stupid idea - totally new ingest pipeline - slimmed down the API (still needs some work0 Closes #235 Closes #340 Closes #360 Closes #361
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
The text was updated successfully, but these errors were encountered: