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

Develop to Master 2021-03-04T13:57:00 #417

Merged
merged 26 commits into from
Mar 5, 2021
Merged

Develop to Master 2021-03-04T13:57:00 #417

merged 26 commits into from
Mar 5, 2021

Conversation

chiragjn
Copy link
Contributor

@chiragjn chiragjn commented Mar 4, 2021

PRs: #391, #395, #393, #416

ranvijayj and others added 26 commits January 12, 2021 01:29
…egory

Updated release drafter to enable Miscellaneous label as a normal category 
Removed miscellaneous from exclude-labels
JIRA TICKET: ADO-4873
Add elastic APM config changes
Add elastic apm to list of packages
Update service name in apm code
Configure WhiteSource Bolt for GitHub
Updated settings.py with requested changes
Updated release drafter to enable Miscellaneous label as a normal category
- Fix http response code for internal server errors for v2/text
- Fix false positive number detection in time detection. Demand a
separator for such cases e.g. fail on "145", e.g. "1:45"
- Update sentry-sdk (and refactor requirements.txt)
- Drop unused static html, js, css files to resolve CVE issues
- Fix IndexError for v1/person_name
- Allow bot message check in person_name to be skipped if data for a
language is missing
- Refactor person_name detector code for multi-lingual
it
Also, change logging formatter, drop NLPLogger, and update setup_sentry
Add elastic APM config changes
Bug fixes for v2/number, v2/number_range, v2/time
@haptik-deployment
Copy link

👍 No lint errors found.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 4, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@haptik-deployment
Copy link

UNIT TESTS HAVE PASSED... Good To Merge

@codecov
Copy link

codecov bot commented Mar 4, 2021

Codecov Report

Merging #417 (a4a6905) into master (e23dc79) will decrease coverage by 30.13%.
The diff coverage is 72.05%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #417       +/-   ##
===========================================
- Coverage   71.16%   41.02%   -30.14%     
===========================================
  Files          78       91       +13     
  Lines        5777     9428     +3651     
===========================================
- Hits         4111     3868      -243     
- Misses       1666     5560     +3894     
Impacted Files Coverage Δ
ner_v1/detectors/textual/name/lang_constants.py 100.00% <ø> (ø)
ner_v2/api.py 12.94% <42.10%> (ø)
...er_v2/detectors/temporal/time/en/time_detection.py 68.96% <60.00%> (+0.50%) ⬆️
...tectors/numeral/number/standard_number_detector.py 91.36% <70.58%> (-0.55%) ⬇️
ner_v1/detectors/textual/name/name_detection.py 80.21% <90.90%> (-0.33%) ⬇️
.../numeral/number_range/en/number_range_detection.py 100.00% <100.00%> (+18.18%) ⬆️
...ral/number_range/standard_number_range_detector.py 93.68% <100.00%> (+3.25%) ⬆️
ner_v2/detectors/textual/tests/test_utils.py
ner_v1/detectors/numeral/size/__init__.py
..._v2/detectors/textual/tests/test_text_detection.py
... and 83 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 e23dc79...a4a6905. Read the comment docs.

@ankur09011 ankur09011 self-requested a review March 5, 2021 05:57
@chiragjn chiragjn merged commit 44b78ec into master Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants