Added comment/debug to damaged FTP file
Doesn't fix issue, but documents what the bug is in the FTP software, and prints the file name and line number that it occurs in so user can fix. Ideally the parser itself would be able to deal with this.
This commit is contained in:
parent
b965654962
commit
a3ff477395
|
@ -202,7 +202,12 @@ class Importer:
|
||||||
if (hand[i].endswith(" has been canceled")): #this is their typo. this is a typo, right?
|
if (hand[i].endswith(" has been canceled")): #this is their typo. this is a typo, right?
|
||||||
cancelled=True
|
cancelled=True
|
||||||
|
|
||||||
seat1=hand[i].find("Seat ") #todo: make this recover by skipping this line
|
#FTP generates lines looking like:
|
||||||
|
#Seat 1: IOS Seat 2: kashman59 (big blind) showed [8c 9d] and won ($3.25) with a pair of Eights
|
||||||
|
#ie. Seat X multiple times on the same line in the summary section, when a new player sits down in the
|
||||||
|
#middle of the hand.
|
||||||
|
#TODO: Deal with this properly, either fix the file or make the parsing code work with this line.
|
||||||
|
seat1=hand[i].find("Seat ")
|
||||||
if (seat1!=-1):
|
if (seat1!=-1):
|
||||||
if (hand[i].find("Seat ", seat1+3)!=-1):
|
if (hand[i].find("Seat ", seat1+3)!=-1):
|
||||||
damaged=True
|
damaged=True
|
||||||
|
@ -216,6 +221,14 @@ class Importer:
|
||||||
partial+=1
|
partial+=1
|
||||||
elif (cancelled or damaged):
|
elif (cancelled or damaged):
|
||||||
partial+=1
|
partial+=1
|
||||||
|
if damaged:
|
||||||
|
print """
|
||||||
|
DEBUG: Partial hand triggered by a line containing 'Seat X:' twice. This is a
|
||||||
|
bug in the FTP software when a player sits down in the middle of a hand.
|
||||||
|
Adding a newline after the player name will fix the issue
|
||||||
|
"""
|
||||||
|
print "File: %s" %(file)
|
||||||
|
print "Line: %s" %(startpos)
|
||||||
else: #normal processing
|
else: #normal processing
|
||||||
isTourney=fpdb_simple.isTourney(hand[0])
|
isTourney=fpdb_simple.isTourney(hand[0])
|
||||||
if not isTourney:
|
if not isTourney:
|
||||||
|
|
Loading…
Reference in New Issue
Block a user