Closed samimak37 closed 3 years ago
Merging #143 (e066252) into master (9bafdee) will increase coverage by
5.11%
. The diff coverage is40.83%
.
@@ Coverage Diff @@
## master #143 +/- ##
==========================================
+ Coverage 51.16% 56.28% +5.11%
==========================================
Files 12 12
Lines 1675 1464 -211
Branches 364 361 -3
==========================================
- Hits 857 824 -33
+ Misses 757 586 -171
+ Partials 61 54 -7
Impacted Files | Coverage Δ | |
---|---|---|
gender_analysis/analysis/dunning.py | 30.00% <0.00%> (ø) |
|
gender_analysis/analysis/dependency_parsing.py | 12.14% <6.25%> (-1.57%) |
:arrow_down: |
gender_analysis/analysis/instance_distance.py | 32.60% <8.69%> (-1.98%) |
:arrow_down: |
gender_analysis/analysis/gender_adjective.py | 45.27% <44.89%> (+14.89%) |
:arrow_up: |
gender_analysis/analysis/gender_frequency.py | 59.33% <57.46%> (+9.69%) |
:arrow_up: |
gender_analysis/gender.py | 100.00% <0.00%> (+2.63%) |
: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 9bafdee...e066252. Read the comment docs.
HUGE WORK all! Everything looks solid to me. I'm going to go ahead and merge.
This PR changes analysis functions in order to use the new
Gender
andPronounSeries
classes in order to un-binarize the analysis functions, along with tackling some other alpha milestones along the way.Gender
andPronounSeries
classes, and allows for a wider variety of user inputfind_gender_adj