It's fine to me if you don't like my approach.
My main issue is the conflicting approaches coming from you. One is the manual's conservative, but still not essentialist, shaping point policy to add some to improve the trace. The other is purge shaping points with a passion: "Remove ALL shaping points!"
The barking order to go far beyond (to the point of going against) what the manual suggests would have been better received (though still not that well received due to the attitude of capital letters and exclamation marks) had it not been coupled with a sensible approach to shaping on other routes adding inconsistency that made me go "huh?"
Certainly I was a bit lazy on the routes where you suggested adding shaping points (and they are added). And certainly I went a little overboard on other routes where you suggest purging all sps.
I remember albsh where sometimes every curve had a wp but you removed a lot of shaping points back then.
I removed a lot mostly as every curve had two shaping points to avoid sharp angle errors and I was pretty much doing every curve. These routes are already much more like the end product - a point at every hairpin bend and maybe one or two others (some because the route would otherwise cross itself) is perhaps a little loose, but it's far more conservative than Albania was. I certainly don't believe I've done every curve here (see CS510 example where I found 67% more curves on a short route!).
Certainly there's scope to remove some shaping points on these routes, and I will reshape them now. But "ALL!" is extreme. As I've established with the length differentials it significantly improves the trace to have at least some of the switchbacking traced.
Looking at CS510:
'every curve' (5sp, 1.31km (119% of stated length))
CG5 http://www.openstreetmap.org/?lat=42.570955&lon=1.486309
+X616645 http://www.openstreetmap.org/?lat=42.569448&lon=1.490756
+X395059 http://www.openstreetmap.org/?lat=42.568000&lon=1.491625
+X676164 http://www.openstreetmap.org/?lat=42.570187&lon=1.492006
+X384310 http://www.openstreetmap.org/?lat=42.570461&lon=1.490804
+X440300 http://www.openstreetmap.org/?lat=42.570722&lon=1.492499
MasRib http://www.openstreetmap.org/?lat=42.568842&lon=1.492209'current file' (3sp, 1.19km (108%))
CG5 http://www.openstreetmap.org/?lat=42.570955&lon=1.486309
+X395059 http://www.openstreetmap.org/?lat=42.568032&lon=1.491523
+X384310 http://www.openstreetmap.org/?lat=42.570461&lon=1.490804
+X440300 http://www.openstreetmap.org/?lat=42.570722&lon=1.492499
MasRib http://www.openstreetmap.org/?lat=42.568842&lon=1.492209'final product' (2sp, 1.04km (95%))
CG5 http://www.openstreetmap.org/?lat=42.570955&lon=1.486309
+X395059 http://www.openstreetmap.org/?lat=42.568000&lon=1.491625
+X440300 http://www.openstreetmap.org/?lat=42.570544&lon=1.491759
MasRib http://www.openstreetmap.org/?lat=42.568842&lon=1.492209I've removed the specific 2 sps from CS310, and two of the four in the section you wanted them all to go: now 7sps instead of 11, and 6.36km (73%) long. I've also removed one pair of hairpins from CS370: now 3sp instead of 5 and 2.81km (88%) long.
Github
