The function `test_re` in the starchat baseline scripts accepts an argument named `re`, but then uses the variable `outp` instead of `re` to get the candidate regex.
Posted by: gbcolborne @ Oct. 13, 2023, 7 p.m.Hello, and thank you very much for pointing this out!
You are right, in the uploaded code `re` should be used in the `test_re` function, rather than `outp`. This seems to have made it into the code during refactoring, we apologise; we have uploaded a new starting kit fixing this issue.
Please note that in this particular case, the use of the wrong variable in `test_re` should still yield the same result as the fixed version: Since we process the generated regular expressions sequentially, using the `outp` variable in the generation loop, at each time we call `test_re` the variable `outp` references the same data that `re` is meant to reference in the function.
Posted by: REIC @ Oct. 18, 2023, 9:08 a.m.There is a similar error in score.py: the score_item function uses the variable header, which is out of scope. It doesn't matter if you call the main function, but it makes it impossible to import the score_item function in other scripts.
Posted by: gbcolborne @ Oct. 18, 2023, 5:38 p.m.You are right, thank you for pointing this out. We've uploaded a new starting kit with a fix.
Posted by: REIC @ Oct. 20, 2023, 8:26 a.m.