Skip to content

Commit 74698e0

Browse files
Merge pull request #6 from nextmv-io/merschformann/fix-links
Fixes broken links
2 parents 85c14cd + 5af706d commit 74698e0

12 files changed

+47
-44
lines changed

.github/workflows/build.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ jobs:
88
runs-on: ubuntu-latest
99
strategy:
1010
matrix:
11-
python-version: ["3.10", "3.11", "3.12"]
11+
# Tests are currently stable only for python 3.11
12+
# due to the way we are testing
13+
python-version: ["3.11"]
1214
steps:
1315
- uses: actions/checkout@v4
1416

README.md

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -125,16 +125,16 @@ Then, add the following line to your `~/.bashrc`:
125125
eval "$(register-python-argcomplete nextplot)"
126126
```
127127

128-
## Development setup
128+
## Tests
129129

130-
### Install development dependencies
130+
Tests are located in `tests/` and can be run via `python -m pytest`. Update test
131+
expectations (golden files) by running `UPDATE=1 python test_cli.py` from the
132+
test directory. Tests require specific versions of the dependencies to be
133+
installed. These can be installed via `pip install -r requirements-dev.txt`.
134+
It is recommended to update the expectations in a docker container to avoid
135+
messing up local dependencies. This can be done by running the following
136+
commands:
131137

132138
```bash
133-
pip install -r requirements-dev.txt
139+
docker run --rm -it -v $(pwd):/app -w /app python:3.11 bash -c "pip install -r requirements-dev.txt && cd tests && UPDATE=1 python test_cli.py"
134140
```
135-
136-
### Tests
137-
138-
Tests are located in `tests/` and can be run via `python -m pytest`. Update test
139-
expectations (golden files) by running `UPDATE=1 python test_cli.py` from the
140-
test directory.

examples/README.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -252,7 +252,7 @@ handle certain data formats. Find an outline of these options here:
252252
(more details) and CartoDB Dark Matter (black to focus colored
253253
routes/cluster). Additional custom map tiles can be added via
254254
`--custom_map_tile [CUSTOM_MAP_TILE]`. An overview can be found here:
255-
https://deparkes.co.uk/2016/06/10/folium-map-tiles/
255+
[folium-tiles][folium-tiles].
256256
- Custom tile providers can even be used by supplying them in the format
257257
`"<tile-url>,<layer-name>,<attribution>"`.
258258
Example:
@@ -261,7 +261,7 @@ handle certain data formats. Find an outline of these options here:
261261
--custom_map_tile "https://{s}.basemaps.cartocdn.com/dark_nolabels/{z}/{x}/{y}{r}.png,DarkMatter no labels,OpenStreetMap authors"
262262
```
263263
264-
(an overview of custom tile providers can be found [here](custom-layers))
264+
(an overview of custom tile providers can be found [here][custom-layers])
265265
- `--jpath_x` & `--jpath_y`:
266266
`nextplot` expects to find arrays of points or indices, if a path like
267267
`state.drivers[*].geometry.coordinates` is given. I.e., either
@@ -300,9 +300,9 @@ handle certain data formats. Find an outline of these options here:
300300
If provided, statistics will be written to the given file in addition to
301301
stdout.
302302
- `rk_bin` (route only):
303-
Path to the [go-routingkit](go-rk) standalone binary. Alternatively,
303+
Path to the [go-routingkit][go-rk] standalone binary. Alternatively,
304304
`routingkit` command will be used at default (requires go-routingkit
305-
[installation](go-rk-install)).
305+
[installation][go-rk-install]).
306306
- `rk_osm` (route only):
307307
Path to the OpenStreetMap data file to be used for routing. All points must be
308308
contained within the region of the file. This file is mandatory when using
@@ -316,3 +316,4 @@ handle certain data formats. Find an outline of these options here:
316316
[go-rk]: https://github.com/nextmv-io/go-routingkit/tree/stable/cmd/routingkit
317317
[go-rk-install]: https://github.com/nextmv-io/go-routingkit/tree/stable/cmd/routingkit#install
318318
[custom-layers]: http://leaflet-extras.github.io/leaflet-providers/preview/
319+
[folium-tiles]: https://deparkes.co.uk/2016/06/10/folium-map-tiles/

requirements-dev.txt

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
argcomplete>=2.0.0
2-
colorutils>=0.3.0
3-
folium>=0.12.1
4-
jsonpath_ng>=1.5.3
5-
kaleido>=0.2.1
6-
numpy>=1.22.3
7-
plotly>=5.7.0
8-
scipy>=1.8.0
9-
pytest>=7.1.1
10-
imagehash>=4.2.1
11-
ruff>=0.1.7
12-
hatch>=1.9.1
1+
argcomplete==2.0.0
2+
colorutils==0.3.0
3+
folium==0.16.0
4+
jsonpath_ng==1.6.1
5+
kaleido==0.2.1
6+
numpy==1.26.4
7+
plotly==5.21.0
8+
scipy==1.13.0
9+
pytest==7.1.1
10+
imagehash==4.3.1
11+
ruff==0.1.7
12+
hatch==1.9.1

tests/testdata/fleet-cloud-comparison.html.golden

Lines changed: 3 additions & 3 deletions
Large diffs are not rendered by default.
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
f0f081e3796a039f
1+
f0f281e3796a031f

tests/testdata/paris-cluster.plot.html.golden

Lines changed: 3 additions & 3 deletions
Large diffs are not rendered by default.

tests/testdata/paris-point.plot.html.golden

Lines changed: 3 additions & 3 deletions
Large diffs are not rendered by default.
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
ff2a8536d604d172
1+
ff228137dc05d172

tests/testdata/paris-route-indexed.plot.html.golden

Lines changed: 3 additions & 3 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)