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

Refactor the pipeline to output multiple reason codes #124

Closed
jeancochrane opened this issue May 23, 2024 · 0 comments
Closed

Refactor the pipeline to output multiple reason codes #124

jeancochrane opened this issue May 23, 2024 · 0 comments
Assignees

Comments

@jeancochrane
Copy link
Collaborator

jeancochrane commented May 23, 2024

Currently, the sales val pipeline outputs reason codes for each flag that combine a characteristic reason with a price reason. For example, Family sale (high) is a combination of a characteristic reason (family sale) with a price reason (high price).

Per the updated sales val design doc, we need to refactor the pipeline so that it outputs multiple reason codes for each flag. This requires us to disaggregate our characteristic reasons from our price reasons, and update the pipeline to output one characteristic reason and one price reason for each flagged sale.

This is what the refactored set of reason codes should look like:

  • Characteristic reasons
    • Short-term owner
    • PTAX-203 Exclusion
    • Family sale
    • Non-person sale
    • Statistical Anomaly
    • Price swing / Home Flip
  • Price reasons
    • High price
    • High price per square foot
    • Low price
    • Low price per square foot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants