Skip to content
Merged
Show file tree
Hide file tree
Changes from 2 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
10 changes: 10 additions & 0 deletions hugo/layouts/report.html
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,16 @@
It is not the same as the Assessor's "comps" tool for reviewing residential appeal
filings, and it is only intended to be used for educational purposes.
</p>
{{ if .Params.special_case_analyst_review }}
<p>
Since your home is a class {{ .Params.char_class }} property, analysts at the
Assessor's office are likely to change its value during the review process.
That's because the market value of this kind of property can depend on income
and expense data that are not included in the statistical model. Because of
this, the characteristics and sales in this report might not explain your
Comment thread
wagnerlmichael marked this conversation as resolved.
Outdated
home's assessed value.
</p>
{{ end }}
</div>

<div class="info-box border">
Expand Down
4 changes: 4 additions & 0 deletions scripts/generate_homeval/generate_homeval.py
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,8 @@ def build_front_matter(
Mapping of variable code for labels and tooltips
"""
special_multi = bool(df_target_pin["is_parcel_small_multicard"].iloc[0])
analyst_review_classes = {"212", "218", "219"}
analyst_review = str(df_target_pin["char_class"].iloc[0]) in analyst_review_classes

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

[Suggestion, required] Preserving some discussion that we had over chat yesterday: It's not ideal that we're pulling this class from the first card in the PIN, given that cards can have different classes, and those classes don't necessarily match up with the PIN-level class. We're going to try switching to PIN-level class here, which will require updating the upstream view in data-architecture to include a pin_class column in pinval.assessment_card.


# Header
tp = df_target_pin.iloc[0] # all cards share the same PIN-level chars
Expand Down Expand Up @@ -222,6 +224,8 @@ def build_front_matter(
"cards": [],
"var_info": vars_dict,
"special_case_multi_card": special_multi,
"special_case_analyst_review": analyst_review,
"char_class": str(df_target_pin["char_class"].iloc[0]),
"environment": environment,
"is_prorated": tp["is_prorated"],
}
Expand Down
Loading