Closed mpadge closed 10 years ago
Mark, looking at
https://github.com/edzer/trajectories/pull/2/files#diff-0
it looks like you tried to improve formatting. As a consequence, you make it too difficult for me to see what has changed.
Please send the pull request with only the changes in it, no reformatting.
On 08/28/2014 01:41 PM, mark padgham wrote:
Edzer, Not yet a perfect solution, but removes the warnings for non par() and plot() args (like arrow lengths). Potential problem remains with propagation of ambiguous args like "col" to Spatial, but this should still be an improvement.
You can merge this Pull Request by running
git pull https://github.com/mpadge/trajectories master
Or view, comment on, or merge it at:
https://github.com/edzer/trajectories/pull/2
Commit Summary
explicit args for plot
File Changes
M R/Tracks-methods.R https://github.com/edzer/trajectories/pull/2/files#diff-0 (54)
Patch Links:
- https://github.com/edzer/trajectories/pull/2.patch
- https://github.com/edzer/trajectories/pull/2.diff
— Reply to this email directly or view it on GitHub https://github.com/edzer/trajectories/pull/2.
Edzer Pebesma Institute for Geoinformatics (ifgi), University of Münster Heisenbergstraße 2, 48149 Münster, Germany. Phone: +49 251 83 33081 http://ifgi.uni-muenster.de GPG key ID 0xAC227795
Edzer, Not yet a perfect solution, but removes the warnings for non par() and plot() args (like arrow lengths). Potential problem remains with propagation of ambiguous args like "col" to Spatial, but this should still be an improvement.