Allow spaces in label (followup)
!24 (merged) fixed the label parsing of lines like Mariam I-Date et
but not of Mariam I-Date et Lieu
. The Regex fails because of the L
in Lieu
. I added a test case for that.
Merge request reports
Activity
changed milestone to %ML Prod - March 2023 n°1
added P2 label
requested review from @starride
Good catch.
I have also an error with the following line
Hello B-surname and L-ocation
.Traceback (most recent call last): File "/home/solene/miniconda3/bin/nerval", line 33, in <module> sys.exit(load_entry_point('nerval', 'console_scripts', 'nerval')()) File "/home/solene/nerval/nerval/cli.py", line 75, in main run(args.annot, args.predict, args.threshold, args.verbose) File "/home/solene/nerval/nerval/evaluate.py", line 308, in run threshold, File "/home/solene/nerval/nerval/evaluate.py", line 99, in compute_matches current_ref.append(char_annot) UnboundLocalError: local variable 'current_ref' referenced before assignment
This is really far-fetched, but we could forbid the
-
character in the labels to avoid this type of errors.
Please register or sign in to reply