Closed j-t-t closed 5 years ago
Merging #220 into master will increase coverage by
<.01%
. The diff coverage is80%
.
@@ Coverage Diff @@
## master #220 +/- ##
==========================================
+ Coverage 54.02% 54.03% +<.01%
==========================================
Files 32 32
Lines 3167 3172 +5
==========================================
+ Hits 1711 1714 +3
- Misses 1456 1458 +2
Open street map has a cycleway tag that can indicate a variety of things, including lane, track, opposite_lane, shared_lane, buffered_lane, etc. Adding these to the model as categorical features.
This does not include curb protected bike paths as they are not indicated on the way itself. That should be done, but is a bit more complex.
Also in this PR is changing oneway and signal to categorical features instead of continuous, which was an unintended change when switching the way we set features.
Once this is merged, a new data-latest will be updated