Skip to content

Feat/2027 edits - #5181

Open
lchen-2101 wants to merge 61 commits into
masterfrom
feat/2027_edits
Open

lchen-2101 wants to merge 61 commits into
masterfrom
feat/2027_edits

Conversation

@lchen-2101

@lchen-2101 lchen-2101 commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

combining all edits into one pr to avoid conflicts.

Comment thread common/src/main/resources/2027EditsDescriptions.txt Outdated
Comment thread hmda/src/main/scala/hmda/validation/rules/lar/validity/2027/V695_5.scala Outdated
Comment thread common/src/main/resources/2027EditsDescriptions.txt
Comment thread common/src/main/resources/2027EditsDescriptions.txt
Comment thread common/src/main/resources/2027EditsDescriptions.txt
val units = config.getInt("edits.Q627.units")

override def apply(lar: LoanApplicationRegister): ValidationResult =
when(lar.property.totalUnits is greaterThanOrEqual(units)) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@chynnakeys the edit looks like it should be between $100,000 and $50,000,000 exclusive, should it be inclusive?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

verbiage is the same compared to previous years', only difference is the higher max limit; previous years' have been inclusive, so if we make this one exclusive, should we make previous years' exclusive as well?

Comment thread hmda/src/main/scala/hmda/validation/rules/lar/quality/_2027/Q650.scala Outdated
Comment thread common/src/main/resources/2027EditsDescriptions.txt Outdated
Comment thread hmda/src/main/scala/hmda/validation/engine/2027/LarEngine2027.scala Outdated
@lchen-2101

Copy link
Copy Markdown
Contributor Author

will add the Q617 division by zero fix in here too.

@lchen-2101
lchen-2101 requested a review from kgudel October 22, 2026 15:00

This branch has not been deployed

No deployments
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

Successfully merging this pull request may close these issues.

2 participants