Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing spaces around NNN-up and NNN-down in survex data #5

Open
goatchurchprime opened this issue Sep 30, 2019 · 1 comment
Open
Assignees
Labels

Comments

@goatchurchprime
Copy link

Missing spaces on this line:
https://github.com/jarvist/migovecsurveydata/blob/master/migovecsurveydata/garden/garden-low/bigrock/miles.svx#L39

26 27 5.39-up;27 is bolt at top of pitch

https://github.com/jarvist/migovecsurveydata/blob/master/migovecsurveydata/garden/garden-low/bigrock/miles.svx#L39

26 27 5.39-up;27 is bolt at top of pitch

(It looks like these are duplicate directories anyway, and probably one of them should be deleted).

This seems to be a common formatting error in a number of places (missing spaces around NNN-up and NNN-down), which prevents tunnel from loading it. Survex handles it, but it's definitely not right, and looks like something that was caused by a script running over the data and rewriting it.

@jarvist
Copy link
Owner

jarvist commented Oct 2, 2019

Ah, nice find + bug dissection!
These errors are there from the instantiation of git. I'll see if I can find the old SVN repo to see who to blame!

I agree it looks like someone applied some kind of shell script to delete all the compass readings from the +-/90 clino legs, to squash those survex warnings. This has been a perennial issue of relatively novice surveyors assuming they should at least try and give a compass direction for the general development of the cave.

@jarvist jarvist self-assigned this Oct 2, 2019
@jarvist jarvist added the bug label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants