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

Splays shouldn't have named stations #31

Open
goatchurchprime opened this issue Jul 28, 2018 · 7 comments
Open

Splays shouldn't have named stations #31

goatchurchprime opened this issue Jul 28, 2018 · 7 comments
Assignees

Comments

@goatchurchprime
Copy link
Contributor

There's a bit of an explosion of station names when previewing in Aven, which is also turning the sections into christmas trees of nodes when you highlight them.

This is because you're using Footleg's top->svx converter which implements the splay legs like this:

1	0	 1.82	137.99	 17.50
*FLAGS SPLAY
1	1a	 1.07	183.25	 70.46
1	1b	 1.14	162.90	-66.71
1	1c	 0.43	160.51	 -9.34
1	1d	 0.78	187.48	-18.34
1	1e	 0.42	 78.82	  2.40
1	1f	 1.06	127.77	  7.12
1	1g	 0.71	114.36	 -1.42
*FLAGS NOT SPLAY

Now, we already have a good way to represent splays as anonymous stations, which you can see from 258/themendips.svx:

x	1	262.85	54.22	3.575	
1	-	108.59	-7.25	1.799	
1	-	134.57	-8.06	2.090	
1	-	187.49	8.71	3.740	
1	-	264.72	25.18	0.576	
1	-	233.34	86.22	5.774	
1	-	234.76	86.42	5.748	
1	-	3.83	5.91	4.166	
1	-	59.98	-43.34	5.518	
1	2	175.47	-9.79	9.976	
2	-	285.47	9.21	1.862	

You'll notice how much easier it is to see how things are connected when you highlight "the_mendips" vs higlighting "mudslope" in Aven.

Is it possible to go back to this format before too many more TOP files get converted? Either by fixing the Footleg converter, or using the Tunnel converter (load the TOP file and copy-paste out the text that is in the big dotted green 'S').

It should be possible to write a script to correct the files already there, to substitute on every line:
"(\d+)\s+(\d+)([a-z])" to "\1 -" when \1 == \2
so the damage is not irreversible. These files are easily identified because they shout "*BEGIN" instead of "*begin" at the start like they've been designed by a FORTRAN programmer -- even though they lack the logical inclusion of "*instrument distox" in them.

Any connection to a splay station is an error.

The only kind of non-stations that should be stations are the proposed QM-type -- which is a GREAT hack, BTW.

@pwithnall
Copy link
Collaborator

Is the source code for Footleg’s converter available anywhere? I can modify it, but I will be doing so blind, as I’m not a paperless surveyor. I can test any changes by grabbing a .top file from expofiles though.

@georgebreley
Copy link
Collaborator

georgebreley commented Jul 29, 2018 via email

@pwithnall
Copy link
Collaborator

pwithnall commented Aug 3, 2018

Thanks George. It appears to be on GitHub too: https://github.com/Footleg/caveconverter

That makes things a bit easier.

@pwithnall
Copy link
Collaborator

Although is it really the right fix to change caveconverter? Aven knows that those legs are splays (from the flags), so why can’t it hide the station names? From the point of view of the Survex file, surely it’s marginally more useful to have identifiers for all the splays than to have them all anonymous? I know I certainly sometimes find it useful to be able to cross-reference legs/splays by ID between plan and elevation views when drawing up.

@pwithnall
Copy link
Collaborator

Anyway, it seems caveconverter already supports anonymous splays, but only in its command line version. The GUI version is hard-coded to use flagged, named splays.

To use anonymous splays with the command line version, it looks like you need to run it something like:

java -jar CaveConverter.jar input.top output.svx p s splays anonsplays

I don’t have the time to dig into the GUI and work out how to add a checkbox for outputting anonymous splays from the GUI, but if you care strongly about that then you could submit an issue against caveconverter.

@pwithnall
Copy link
Collaborator

Alternatively, if there’s a strong rationale for using anonymous splay stations in caveconverter (other than “it’s easier to change caveconverter than it is to change Aven”), it would be fairly straightforward to change the GUI so it always outputs anonymous splay stations rather than named ones. Presumably Footleg chose to have them named by default for a reason, though.

@AndrewAkinson
Copy link

AndrewAkinson commented Aug 4, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants