search

CUNY-CL / latin_scansion

Apache License 2.0
0 stars 2 forks source link

Adds escaping before norm rewriting #38

Closed kylebgorman closed 3 years ago

kylebgorman commented 3 years ago

Pynini treats [ and ] as enclosing a generating symbol unless they are "escaped" immediately before the rewrite.

Adds a test to check this behavior, also.

jillianchang commented 3 years ago

Actually, I'm getting this error

Traceback (most recent call last):
  File "/Users/jillianchang/LatinScansion/src/scan.py", line 39, in <module>
    main(parser.parse_args())
  File "/Users/jillianchang/LatinScansion/src/scan.py", line 20, in main
    lines = [line.rstrip() for line in lines]
UnboundLocalError: local variable 'lines' referenced before assignment
kylebgorman commented 3 years ago

Oof, I'll fix that. I know what I did.

On Thu, Aug 12, 2021 at 5:56 PM jillianchang @.***> wrote:

Actually, I'm getting this error

Traceback (most recent call last): File "/Users/jillianchang/LatinScansion/src/scan.py", line 39, in main(parser.parse_args()) File "/Users/jillianchang/LatinScansion/src/scan.py", line 20, in main lines = [line.rstrip() for line in lines] UnboundLocalError: local variable 'lines' referenced before assignment

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/CUNY-CL/LatinScansion/pull/38#issuecomment-897993995, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABG4OO7IM5RHNRQMFSWYNLT4Q7R3ANCNFSM5CCEAGGA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .