Closed kenalba closed 3 years ago
Merging #149 (15d20eb) into master (711a51e) will decrease coverage by
0.66%
. The diff coverage is39.76%
.
@@ Coverage Diff @@
## master #149 +/- ##
==========================================
- Coverage 56.40% 55.73% -0.67%
==========================================
Files 12 13 +1
Lines 1468 1552 +84
Branches 362 379 +17
==========================================
+ Hits 828 865 +37
- Misses 586 631 +45
- Partials 54 56 +2
Impacted Files | Coverage Δ | |
---|---|---|
gender_analysis/analysis/gender_pos.py | 39.18% <39.18%> (ø) |
|
gender_analysis/analysis/gender_adjective.py | 80.00% <50.00%> (+34.72%) |
:arrow_up: |
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 711a51e...15d20eb. Read the comment docs.
@kenalba Thanks for getting started on this work. I've picked up the project with a few modifications (creating a single interface for passing in any NLTK tokens into the new GenderProximityAnalyzer
class) in PR #159, which is now merged into the main branch. If it's alright with you, I'll close this PR.
Generalized the gender_adjective function with gender_pos. An open question is whether I've dealt with the now-semi-vestigal gender_adjective function, most of whose functions now just point at gender_pos. All of the tests pass, in any case!