voila-gpx-viewer environment settings upgraded to new and still supported packages #28
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… code that dynamically show the speed in a display!
…e needs to be converted into float number!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The voila-gpx-viewer you have written seems not to be supported anymore, as I remember it was properly working few months ago and it does not work in none of the two newly installed Anaconda distributions I recently used on my laptop (I've used an Ubuntu Linux distribution before and now a Linux Mint one).
Particularly, running the "environment.yml" file via conda, nothing is going to be upgraded anymore and the software does not work at all.
Basically, the job I did here makes voila-gpx-viewer available again, also on newly installed Anaconda distributions.
To do that, I've added a "inst.sh" file which basically makes the job of the "environment.yml" (which is still present in between the files and I believe it could be deleted) you've written before.
In the README.md file, also modified, I explain how to set up the new environment using the "inst.sh file; the "environment.yml" can be ignored at all.
Mostly, the difference between my "inst.sh" and your "environment.yml" file is that I've replaced the "conda" lines of code with "pip" ones. Furthermore the "inst.sh" file provides packages versions which are not the some as the "environment.yml" ones.
Particularly, the altimetry data seem to come from a web source which is not available anymore on the some link as before. For this reason I've repleced the srtm.py package with the upgraded one, which is 0.3.7 and takes altimetry data from the right links (without showing any message of error!).
Consider this is my second Pull request ever. The first one I did almost one year ago, where I was training on Git and GitHub and you gently accepted my request of doing my first pull request ever.
Fill free to suggest me any further modification.
Waiting for a your answer, I thank you for your attention