Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace source of link to CCAO class dict in intro vignette #45

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion vignettes/introduction.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ print(bill)
|-------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
| **`year`** | Tax year of the bill. Note that bills are paid in arrears, so a 2021 bill is paid in 2022 | Must be >= 2006 |
| **`pin`** | Property Index Number of the property. This is the unique identifier of each parcel in Cook County | |
| **`class`** | The 3-digit Assessor minor class code. This identifies the property type/structure | See [class dictionary](https://github.com/ccao-data/ccao/blob/master/data-raw/class_dict.csv) |
| **`class`** | The 3-digit Assessor minor class code. This identifies the property type/structure | See [class dictionary](https://github.com/ccao-data/data-architecture/blob/master/dbt/seeds/ccao/ccao.class_dict.csv) |
| **`tax_code`** | The 5-digit tax code. Each tax code identifies the unique combination of taxing districts that a PIN is within. Note that the first 2 digits of `tax_code` are the Assessor township code | |
| **`av`** | Assessed value of the property. By default, this is the finalized AV *after all appeals have been completed*. See [optional arguments](#optional-arguments) below for more details | Must be >= 0 |
| **`eav`** | The `av` multiplied by the state equalization factor | Must be >= 0 |
Expand Down
7 changes: 3 additions & 4 deletions vignettes/mapping.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -218,10 +218,9 @@ Sometimes you may need to create a map of a non-taxing area like a ward, judicia
To demonstrate such a case, we can use Chicago Ward 48. The first thing we need to do is find a boundary of Ward 48. Fortunately, the ward boundaries are available on the [Chicago Data Portal](https://data.cityofchicago.org/Facilities-Geographic-Boundaries/Boundaries-Wards-2015-2023-/sp34-6z76).

```{r, message=FALSE, warning=FALSE, results='hide'}
ward_bound <- st_read(paste0(
"https://data.cityofchicago.org/api/geospatial/",
"sp34-6z76?method=export&format=GeoJSON"
)) %>%
ward_bound <- st_read(
"https://data.cityofchicago.org/resource/k9yb-bpqx.geojson"
) %>%
Comment on lines +221 to +223
Copy link
Contributor Author

@jeancochrane jeancochrane Dec 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated change I noticed while running tests on CI: It seems like the old URL no longer works. You can confirm this by visiting the non-export version of the URL, which returns no data: https://data.cityofchicago.org/api/geospatial/sp34-6z76

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

praise: Nice catch!

filter(ward == "48")
```

Expand Down
Loading