ChHarding / TouchTerrain_for_CAGEO

Touch Terrain: A python app to create 3D printable terrain models (STL/OBJ) from only elevation data (via Google Earth Engine) or from a local geotiff. Has been used for CNC terrain models. Runs as a web app (http://touchterrain.org), as .py file (standalone.py) or as jupyter notebook. Docker image: https://github.com/ChHarding/TouchTerrain_jupyter_docker
http://touchterrain.geol.iastate.edu
210 stars 45 forks source link

Inconsistent GPX track width #76

Open atrevarrow opened 1 year ago

atrevarrow commented 1 year ago

When adding a gpx track, the algorithm for stacking multiple lines to produce a single thick track produces inconsistent results depending on the direction of the line.

For example:

image

As you can see, NW-SE portions are very thin (no added thickness) whereas SW-NE parts are fine.

I have created a PR to improve this: https://github.com/ChHarding/TouchTerrain_for_CAGEO/pull/75

With the new version, the same track looks like this:

image

Still not perfect, but I think an improvement.

(BTW, thanks for creating this open-source project 👍)

ChHarding commented 1 year ago

Thank you very much! I’ll put it into the next version of TouchTerrain and will give you credit in the patch notes.

On Mar 22, 2023, at 22:30, atrevarrow @.**@.>> wrote:

When adding a gpx track, the algorithm for stacking multiple lines to produce a single thick track produces inconsistent results depending on the direction of the line.

For example:

[image]https://user-images.githubusercontent.com/5221201/227093794-66876ec5-d677-4894-8426-02f936e53020.png

As you can see, NW-SE portions are very thin (no added thickness) whereas SW-NE parts are fine.

I have created a PR to improve this: #75https://github.com/ChHarding/TouchTerrain_for_CAGEO/pull/75

With the new version, the same track looks like this:

[image]https://user-images.githubusercontent.com/5221201/227094629-0e0b7764-fe08-4625-b6d4-a316ce4e7fe0.png

Still not perfect, but I think an improvement.

(BTW, thanks for creating this open-source project 👍)

— Reply to this email directly, view it on GitHubhttps://github.com/ChHarding/TouchTerrain_for_CAGEO/issues/76, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEYDF5JJZPKLYH7OWZLNK2TW5O7VVANCNFSM6AAAAAAWEUPE6M. You are receiving this because you are subscribed to this thread.Message ID: @.***>

Chris Harding Associate Professor Department of Geological & Atmospheric Sciences Touchterrain.geol.iastate.eduhttp://Touchterrain.geol.iastate.edu