error when running the tutorial example about persephone

[shuttle1987]

I'm thinking that we can change the assert here to a warning based on a floating point close-equals that prints out a message to the user.

[shuttle1987]

Thanks for the report, I'll get this fixed soon

[tensorfoo]

assert test_per == test_ler

the assertion being failed on line 293 of model.py

[shuttle1987]

Thanks for opening this issue.

@oadams I'd appreciate a comment about the intent behind the code that contains the assertion here:

persephone/persephone/model.py

Lines 293 to 296 in b4ce92f

	assert test_per == test_ler
	with open(os.path.join(hyps_dir, "test_per"), "w",
	encoding=ENCODING) as per_f:
	print("LER: %f" % (test_ler), file=per_f)

[oadams]

The assertion would have been to ensure that the tensorflow label error rate was consistent with what was measured by my implementation of minimum edit distance at some point. But it looks like utils.batch_per() is using the NLTK implementation of edit distance anyway. It could be a floating point precision thing, but I'd like to see what the actual values are since if that's the case it's a bit surprising this hasn't popped up so far.

Either way, the assert should probably go since it doesn't help anyone.

[tensorfoo]

It could be a floating point precision thing, but I'd like to see what the actual values are since if that's the case it's a bit surprising this hasn't popped up so far.

Yup! Good intuition. I ran the training again and had it print the two variables just before the assertion:
test_per= 0.2878205128205128 test_ler= 0.28782052

I've also got it paused on the debugger on the assertion line if you guys want me to explore further, please let me know soon. For the record i've hit this bug 3/3 times i've run the training, so it seems pretty reproducible.

[shuttle1987]

@tensorfoo Hopefully the changes I made in #224 resolve your issue, if not please let us know