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.

Sign up for GitHub

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

Jump to bottom

0.5.5 #134

Merged
merged 93 commits into from
Sep 1, 2023
Merged

0.5.5 #134

merged 93 commits into from
Sep 1, 2023

Conversation

yiwen-h
Copy link
Member

@yiwen-h yiwen-h commented Aug 29, 2023

Major improvements include:

  • Introduction of Ruff linting
  • Docker container containing transformer-based sentiment model
  • Improved test coverage
  • Added datasets for phase 2
  • Enabled enhancement of predicted probabilities using specific vocabulary

yiwen-h and others added 30 commits June 16, 2023 08:59
@yiwen-h
added ruff
402faea
@yiwen-h
autofix with ruff
e2b821e
@yiwen-h
added codecov to test gh action
133473f
@yiwen-h
added ruff to toml
62cd264
@yiwen-h
fixed unused loop variable in for loop
5f0732d
@yiwen-h
fixed errors identified by ruff in multilabel_pipeline
3944e40
@yiwen-h
fixed ruff identified errors in factory_data_load_and_split
b3908c2
@yiwen-h
fixed ruff error in factory_model_performance
50ee694
@yiwen-h
fixed minor ruff identified errors in factory_predict_unlabelled_text…
a252b19 
… and sentiment_pipeline
@yiwen-h
adding precommit
bea9387
@yiwen-h
added folders to exclude in precommit
5aa23b2
@yiwen-h
fixing test GH action yaml
c0a6627
@yiwen-h
updated precommit yaml
237bc4f
@yiwen-h
added test_sklearn_pipeline_sentiment
6f240ab
@yiwen-h
added ruff to GH action
9aa28a6
@yiwen-h
modified workflow yaml so upload to codecov only happens once
badc9ac
@yiwen-h
added more tests for factory_pipeline
773e812
@yiwen-h
added test_process_data
200dd0b
@yiwen-h
removed tf pipeline and functions
470bafc
@yiwen-h
added test_create_sklearn_pipeline
b25864e
@yiwen-h
added test_search_sklearn_pipelines
d6c8596
@yiwen-h
added more tests for factory_pipeline
437e6d1
@yiwen-h
Merge pull request #114 from CDU-data-science-team/104-75_linting_and…
cc0762d 
…_tests

104 75 linting and tests
@yiwen-h
added tests for factory_model_performance
2a4c160
@yiwen-h
Merge pull request #116 from CDU-data-science-team/75_pipeline_tests
0b9e304 
added tests for factory_model_performance
@yiwen-h
started basic framework for dockerizing sentiment endpoint
8556d2b
@yiwen-h
working docker container - cant use Alpine
243c69b
@yiwen-h
got docker container to mount data folder, accept filename as argument
3ad8e6b
@yiwen-h
added test_sklearn_pipeline
7885fcc
@yiwen-h
parametrized target for sklearn pipeline. wrote svc pipeline test
4fdd06c
yiwen-h and others added 25 commits August 9, 2023 14:54
@yiwen-h
fixed ruff complaining about == instead of isinstance in tests
40a3cfd
@yiwen-h
fixed ruff complaining about == instead of isinstance in test_factory…
f1256ed 
…_pipeline
@yiwen-h
some broken dependencies causing test to fail, trying to fix pyprojec…
301c8b7 
…t.toml
@yiwen-h
Merge pull request #131 from CDU-data-science-team/126_ROC
c815088 
126 roc
@yiwen-h
started basic framework for dockerizing sentiment endpoint
18bf54d
@yiwen-h
working docker container - cant use Alpine
d4f0831
@yiwen-h
got docker container to mount data folder, accept filename as argument
784c306
@yiwen-h
json input file now deleted if NOT run locally
98c1846
@yiwen-h
Predictions now outputted as json file in data_out folder
72c5a24
@yiwen-h
added label to dockerfile
1a30581
@yiwen-h
added most tests for docker_run
dfe0907
@yiwen-h
added larger json file - about 8000 comments
f3549b0
@yiwen-h
added cache removal to dockerfile in bid to reduce size
12a0f79
@yiwen-h
fewer layers, slim-debian, for smaller size
eb94f9a
@yiwen-h
updated dockerfile to reduce size
866154c
@yiwen-h
mocking load_model in test_predict_sentiment
ca2e483
@yiwen-h
Merge pull request #132 from CDU-data-science-team/118_distilbert_API
5894def 
118 distilbert api
@yiwen-h
updated packages using poetry to address dependabot security issues
c3a0c7f
@yiwen-h
updated api requirements to match 0.5.5 package requirements
f35e007
@yiwen-h
added proper reqs file to api folder (with dev group)
71ece91
@ChrisBeeley
Merge pull request #133 from CDU-data-science-team/security_fixes_16_…
0c0e7e5 
…17_25_26

Security fixes 16 17 25 26
@yiwen-h
added more docstrings
60e1415
@yiwen-h
updated docs for API page and sentiment_pipeline
44ac72b
@yiwen-h
added tornado^6.3.3 to address security issue #27
955e42f
@yiwen-h
updated version to 0.5.5
bf43ede
@yiwen-h yiwen-h merged commit 95e56e0 into main Sep 1, 2023
3 checks passed
@yiwen-h yiwen-h deleted the 0.5.5 branch September 1, 2023 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChrisBeeley ChrisBeeley ChrisBeeley approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yiwen-h @ChrisBeeley