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

75 finaltests #143

Merged
merged 12 commits into from
Sep 25, 2023
Merged

75 finaltests #143

merged 12 commits into from
Sep 25, 2023

Conversation

yiwen-h
Copy link
Contributor

@yiwen-h yiwen-h commented Sep 22, 2023

Got testing up to 99%, the last few bits not tested will be removed with issues #142 and #141

@codecov
Copy link

codecov bot commented Sep 22, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +10.17% 🎉

Comparison is base (5c1c217) 88.78% compared to head (4bf927e) 98.96%.
Report is 13 commits behind head on development.

Additional details and impacted files
@@               Coverage Diff                @@
##           development     #143       +/-   ##
================================================
+ Coverage        88.78%   98.96%   +10.17%     
================================================
  Files               11       11               
  Lines              945      962       +17     
================================================
+ Hits               839      952      +113     
+ Misses             106       10       -96     
Files Changed Coverage Δ
...extmining/factories/factory_data_load_and_split.py 100.00% <100.00%> (+5.78%) ⬆️
pxtextmining/factories/factory_pipeline.py 100.00% <100.00%> (+7.05%) ⬆️
...ining/factories/factory_predict_unlabelled_text.py 100.00% <100.00%> (+16.83%) ⬆️
pxtextmining/pipelines/multilabel_pipeline.py 100.00% <100.00%> (+11.45%) ⬆️
pxtextmining/pipelines/sentiment_pipeline.py 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yiwen-h
Copy link
Contributor Author

yiwen-h commented Sep 22, 2023

Finishes and closes #75

@yiwen-h yiwen-h merged commit 9d48b20 into development Sep 25, 2023
5 checks passed
@yiwen-h yiwen-h deleted the 75_finaltests branch September 25, 2023 07:54
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.

None yet

1 participant