X-DataInitiative / SCALPEL-Extraction

SCALPEL extraction library to fetch concepts from flattened SNDS
BSD 3-Clause "New" or "Revised" License
17 stars 3 forks source link

Cnam-303-fractures-definition-update #126

Closed ysebiat closed 6 years ago

ysebiat commented 6 years ago

This pull request has two folds:

  1. It applies new filters to MCO et MCE_CE source reading.
  2. It applies an update on the definition of Liberal & Private Ambulatory fractures.
codecov[bot] commented 6 years ago

Codecov Report

Merging #126 into master will decrease coverage by 0.38%. The diff coverage is 90.32%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #126      +/-   ##
==========================================
- Coverage   86.51%   86.13%   -0.39%     
==========================================
  Files         108      110       +2     
  Lines        1668     1716      +48     
  Branches       53       36      -17     
==========================================
+ Hits         1443     1478      +35     
- Misses        225      238      +13
Impacted Files Coverage Δ
...ique/cmap/cnam/etl/sources/value/IrImbSource.scala 100% <ø> (ø)
...ap/cnam/etl/extractors/acts/McoCEMedicalActs.scala 100% <ø> (ø) :arrow_up:
...ue/cmap/cnam/etl/sources/value/DosagesSource.scala 100% <ø> (ø)
...ue/cmap/cnam/etl/extractors/acts/MedicalActs.scala 100% <ø> (ø) :arrow_up:
...hnique/cmap/cnam/etl/sources/data/DcirSource.scala 100% <ø> (ø)
...map/cnam/etl/sources/data/McoSourceSanitizer.scala 100% <100%> (ø)
...hnique/cmap/cnam/etl/sources/data/McoFilters.scala 100% <100%> (ø)
...chnique/cmap/cnam/etl/sources/data/McoSource.scala 100% <100%> (ø)
.../polytechnique/cmap/cnam/etl/sources/Sources.scala 69.56% <100%> (ø) :arrow_up:
...nique/cmap/cnam/etl/sources/data/McoCeSource.scala 100% <100%> (ø)
... and 8 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 c4c998e...4f70de8. Read the comment docs.

danielpes commented 6 years ago

@ysebiat Some lines are not being hit by tests. You can check them here to verify if there's a good reason for this.