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

Applicant Income Race and Ethnicity Report #2701

Merged
merged 7 commits into from
Apr 25, 2019

Conversation

kgudel
Copy link
Contributor

@kgudel kgudel commented Apr 22, 2019

This PR makes a small change to the json structure from last year in the borrower characteristics section:

The 2017 format:
2017Json

Becomes:

2018Json

@codecov
Copy link

codecov bot commented Apr 22, 2019

Codecov Report

Merging #2701 into master will decrease coverage by 0.6%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2701      +/-   ##
==========================================
- Coverage   57.85%   57.25%   -0.61%     
==========================================
  Files         545      547       +2     
  Lines       10073    10234     +161     
  Branches      275      295      +20     
==========================================
+ Hits         5828     5859      +31     
- Misses       4245     4375     +130
Impacted Files Coverage Δ
...main/scala/com/hmda/reports/AggregateReports.scala 0% <ø> (ø) ⬆️
...a/com/hmda/reports/processing/BaseProcessing.scala 0% <ø> (ø) ⬆️
.../modifiedlar/ModifiedLoanApplicationRegister.scala 0% <ø> (ø) ⬆️
...hmda/reports/processing/DisclosureProcessing.scala 0% <ø> (ø) ⬆️
.../hmda/query/repository/ModifiedLarRepository.scala 0% <ø> (ø) ⬆️
...ain/scala/com/hmda/reports/DisclosureReports.scala 0% <ø> (ø) ⬆️
...rts/processing/IncomeRaceEthnicityProcessing.scala 0% <0%> (ø)
.../publication/lar/parser/ModifiedLarCsvParser.scala 0% <0%> (ø) ⬆️
.../hmda/reports/processing/AggregateProcessing.scala 0% <0%> (ø) ⬆️
.../publication/lar/parser/IncomeCategorization.scala 0% <0%> (ø)
... and 38 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e941982...e1fcce6. Read the comment docs.

@codecov
Copy link

codecov bot commented Apr 22, 2019

Codecov Report

Merging #2701 into master will decrease coverage by 0.93%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2701      +/-   ##
==========================================
- Coverage   56.85%   55.91%   -0.94%     
==========================================
  Files         546      548       +2     
  Lines       10251    10419     +168     
  Branches      276      277       +1     
==========================================
- Hits         5828     5826       -2     
- Misses       4423     4593     +170
Impacted Files Coverage Δ
...main/scala/com/hmda/reports/AggregateReports.scala 0% <ø> (ø) ⬆️
...a/com/hmda/reports/processing/BaseProcessing.scala 0% <ø> (ø) ⬆️
.../modifiedlar/ModifiedLoanApplicationRegister.scala 0% <ø> (ø) ⬆️
...hmda/reports/processing/DisclosureProcessing.scala 0% <ø> (ø) ⬆️
.../hmda/query/repository/ModifiedLarRepository.scala 0% <ø> (ø) ⬆️
...ain/scala/com/hmda/reports/DisclosureReports.scala 0% <ø> (ø) ⬆️
...rts/processing/IncomeRaceEthnicityProcessing.scala 0% <0%> (ø)
.../publication/lar/parser/ModifiedLarCsvParser.scala 0% <0%> (ø) ⬆️
.../hmda/reports/processing/AggregateProcessing.scala 0% <0%> (ø) ⬆️
.../publication/lar/parser/IncomeCategorization.scala 0% <0%> (ø)
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d56107...d8ad7c7. Read the comment docs.

@kgudel kgudel changed the title WIP: Applicant Income Race and Ethnicity Report Applicant Income Race and Ethnicity Report Apr 23, 2019
@omnipresent07
Copy link
Contributor

LGTM 👍

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