vertex-related cleanup

[yakra]

siteupdate.py
# compress edges adjacent to hidden vertices
for label, v in self.vertices.items():

• label unused. Is actually a Waypoint now.
• consider w instead of v.first_waypoint

->

HGVertex::first_waypoint no longer used (aleady unused in C++).
Was it before I started messing with the datachecks?

[yakra]

siteupdate.py
# compress edges adjacent to hidden vertices
for label, v in self.vertices.items():

• label unused. Is actually a Waypoint now.
• consider w instead of v.first_waypoint

3f938cb

---

HGVertex::first_waypoint no longer used (aleady unused in C++).

DataProcessing/siteupdate/python-teresco/siteupdate.py

Lines 1692 to 1693 in 79bf1cd

	# note: if saving the first waypoint, no longer need
	# lat & lng and can replace with methods

DataProcessing/siteupdate/cplusplus/classes/GraphGeneration/HGVertex.cpp

Lines 22 to 23 in df1ba5d

	// note: if saving the first waypoint, no longer need
	// lat & lng and can replace with methods

We can remove either lat & lng or first_waypoint, but not both.

• RAM:
  • C++: can save just under 8MB or just under 4MB respectively.
  • Py: I assume the savings will be a bit greater, since we need to store an object's type as well as its location.
• Speed:
  • C++ is a fraction of a second slower with a method, probably due to more dereferencing.
  • Py: 77.3 s, up from 74.77

---

Related: #153
If removing lat & lng, squash.
If removing first_waypoint, just do it.

---

Was it before I started messing with the datachecks?

May end up not caring about this question enough to answer it. :)

[yakra]

We can remove either lat & lng or first_waypoint, but not both.

Removing lat & lng and replacing with a method is slower. Also requires some (minor) changes to PlaceRadius.contains_vertex in Python. Removing first_waypoint instead.

dead branches

on branch	last commit	descr.
HGVertex	e21946d996ace80530c9d92b798db65b9f931dc4	C++ HGVertex lat/lng methods Slower. Hard reset.
HGVertexPy	b5a79f5f1ce71c982f5940181ca9f8ce737a4348	Python HGVertex lat/lng methods PlaceRadius.contains_vertex bugfix Slower. Delete branch.