Skip to content

Improvement for MLChain 0.1.9#30

Merged
meocong merged 31 commits intomasterfrom
sentry
Dec 29, 2020
Merged

Improvement for MLChain 0.1.9#30
meocong merged 31 commits intomasterfrom
sentry

Conversation

@lamhoangtung
Copy link
Copy Markdown
Contributor

@lamhoangtung lamhoangtung commented Dec 29, 2020 •
edited
Loading

lamhoangtung and others added 30 commits September 29, 2020 18:19
@lamhoangtung
Resolve merge conflict
04c9ff7
@lamhoangtung
Compress CSS
67f5b2b
@lamhoangtung
Compressed js
9578173
@lamhoangtung
Compress more html
68330e3
@lamhoangtung
Detached SVG and compress final HTML file
f176e47
@lamhoangtung
potential fix for #20
4745975
@lamhoangtung
Minor pipeline bug fixed
8277e0b
@lamhoangtung
Minor coverage test increase
add380c
@lamhoangtung
Merge branch 'master' into linus_dev
d2d8c04
@lamhoangtung
More test for more coverage
a190f05
@lamhoangtung
Raise some human readable exception when using CLI
1e21c57
@lamhoangtung
fixed #19
02bfc83
@meocong
🔖 Update mlconfig 🔖
5c3c8cc 
Env var is now on top of priority in mlconfig
@meocong
🚀 🚀 🚀 Update mlchain 0.1.9 🚀 🚀 🚀
7edbe8d 
- Sentry integration. Fixed #28
- Update better interface for mlconfig.yaml
- Put env on the most priority
- Handle some exceptions
@meocong
Update requirements + Fixed sentry when no dsn
a364bf8
@meocong
🐛 Higher Priority for Sentry config using env vars + Fixed requiremen…
00b1db5 
…ts for Python 3.6 🐛
@meocong
🎉 🎉 🎉 Handle exception + Fixed Sentry break Gunicorn + Better default…
cff1f4c 
… config🎉 🎉 🎉
@lamhoangtung
Merge branch 'linus_dev' into sentry
8f3c1e6
@meocong
🐛 Force value of version to sentry is string
b5535e9
@meocong
Merge branch 'sentry' of https://github.com/Techainer/mlchain-python …
d4f1c4b 
…into sentry
@meocong
🐛 Improved capture_exception sentry + Fixed config and mlchain run
588c8ac
@meocong
🐛 Fixed handle client exception with new httpx
ac524ef
@meocong
Better handle exception with sentry
1b5a1c5
@meocong
👕 Update MlchainError work with sentry
f9f51ba
@meocong
✨ Change transaction id into UUID4 ✨
bcd9073
@meocong
✨ Better handle exception when mlchain run
93e7ab4
@lamhoangtung
Minor improvement for non sentry initialization
237d74a
@lamhoangtung
Minor bug fixed
50f3bf8
@lamhoangtung
Minor function to set mlchain context id
f011f02
@lamhoangtung
Init get GPU stats once
dd28a14
@lamhoangtung lamhoangtung requested a review from meocong December 29, 2020 00:30
@lamhoangtung lamhoangtung added the enhancement New feature or request label Dec 29, 2020
@lamhoangtung lamhoangtung self-assigned this Dec 29, 2020
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 29, 2020 •
edited
Loading

Codecov Report

Merging #30 (bde3fc5) into master (12c52cd) will increase coverage by 5.93%.
The diff coverage is 56.69%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
+ Coverage   49.46%   55.39%   +5.93%     
==========================================
  Files          44       47       +3     
  Lines        3613     3809     +196     
==========================================
+ Hits         1787     2110     +323     
+ Misses       1826     1699     -127
Impacted Files Coverage Δ
mlchain/base/serve_model.py 51.68% <ø> (+18.34%) ⬆️
mlchain/base/wrapper.py 33.33% <0.00%> (-1.67%) ⬇️
mlchain/cli/main.py 100.00% <ø> (+6.66%) ⬆️
mlchain/cli/mlchain_server.py 0.00% <0.00%> (ø)
mlchain/server/__init__.py 100.00% <ø> (+56.25%) ⬆️
mlchain/server/flask_server.py 56.38% <0.00%> (+0.73%) ⬆️
mlchain/server/grpc_server.py 35.08% <0.00%> (ø)
mlchain/server/quart_server.py 26.23% <0.00%> (+0.29%) ⬆️
mlchain/cli/run.py 62.86% <41.66%> (-2.44%) ⬇️
mlchain/server/view.py 62.41% <50.53%> (+2.91%) ⬆️
... and 30 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 12c52cd...bde3fc5. Read the comment docs.

@meocong meocong merged commit e48cc3a into master Dec 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@meocong meocong meocong approved these changes

Assignees

@lamhoangtung lamhoangtung

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

2 participants

@lamhoangtung @meocong