Remove visited as the info is in colormap

pszufe / OpenStreetMapX.jl

OpenStreetMap (*.osm) support for Julia 1.0 and up

MIT License

123 stars 24 forks source link

Remove visited as the info is in colormap #53

Closed blegat closed 3 years ago

blegat commented 3 years ago

We have the relation visited[u] == !iszero(colormap[u]) so visited does not carry any additional information

pszufe commented 3 years ago

again conflicts here. coveralls also complains about coverage increase - adding a single unit test for this change would have been nice

blegat commented 3 years ago

The lines seems to be covered by the tests. The coverage decreased because the PR removed lines that were covered so the % of covered lines decreased.