Some confusion over Orphanet identifiers in EFO

[gaurav]

The EFO code has two pieces of code that appear to be trying to exclude Orphanet identifiers from the concords generated from EFO:

Babel/src/datahandlers/efo.py

Lines 103 to 107 in e9830ed

	if otherid.startswith("ORPHANET"):
	print(row["match"])
	print(other)
	print(otherid)
	exit()

Babel/src/datahandlers/efo.py

Lines 125 to 129 in e9830ed

	if otherid.startswith("ORPHANET"):
	print(row["match"])
	print(other)
	print(otherid)
	exit()

However, because these methods call exit(), it looks to me like they used to stop generating output when they hit their first Orphanet identifier? Secondarily, because this code didn't standardize capitalization, it was probably missing many (all?) Orphanet identifiers anyway.

I've updated that code in PR #402 so that we skip Orphanet identifiers instead, and so now these identifiers are being skipped.

So we need to figure out:

1. What the original intention of this code was, and
2. Whether we want to skip Orphanet identifiers going forward