BUS_WITH_I datacheck

[yakra]

DataProcessing/siteupdate/python-teresco/siteupdate.py

Lines 3349 to 3351 in 5c69952

	# look for I-xx with Bus instead of BL or BS
	if re.fullmatch('I\-[0-9]*Bus', w.label):
	datacheckerrors.append(DatacheckEntry(r,[w.label],'BUS_WITH_I'))

requires a full match, which means some labels (e.g. with directional suffixes, or abbrevs) could slip thru the cracks.
I'll cook up a few test cases and test them out.

C++ version is not affected.

[yakra]

Overlooks route "numbers" such as 35W, 35E, 69W, 69C, 69E

[yakra]

Case sensitive = potential for false negatives. How about all regexes in siteupdate.py?
Their C++ equivalents?

[yakra]

Only flag these when country is USA.

Potential for FPs:
usai;TX;I-69;;Bus;Buster Keaton, TX;tx.i069bus; ;)
Solution: check for intersecting route; abbrev

[yakra]

requires a full match, which means some labels (e.g. with directional suffixes, or abbrevs) could slip thru the cracks.

Just slap a .* at the end and we're good to go.

task	Py	C++
require >= 1 numeral	Done	Done
account for [NEWS] suffixes	Done	Done
case insensitive Bus	Done	Done
don't require full match	Done	Done

[yakra]

Only flag these when country is USA.

• Easy enough in the C++ version.
• The Python version has no region class, so... grumble, I guess turn all_regions into a dict, keys = region codes, values = the existing lists of fields. Where we now iterate through all_regions, we can iterate through all_regions.values(). A simple enough conversion.

Potential for FPs:
usai;TX;I-69;;Bus;Buster Keaton, TX;tx.i069bus; ;)
Solution: check for intersecting route; abbrev

Nope. Because the C++ version performs this datacheck while reading .wpt files from disk, there's potential for *colocated to not be populated yet. It's not worth making another pass thru the data to do this. On the rare theoretical occasion this would occur, it can just be marked FP.