Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate pylint error #463

Closed
niknetniko opened this issue Nov 24, 2023 · 1 comment · Fixed by #466
Closed

Investigate pylint error #463

niknetniko opened this issue Nov 24, 2023 · 1 comment · Fixed by #466
Labels
bug Something isn't working

Comments

@niknetniko
Copy link
Member

Standard Error:

Pylint crashed with
Traceback (most recent call last):
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/languages/python/linter.py", line 46, in run_pylint
    lint.Run(args, reporter=JSONReporter(output=pylint_out), do_exit=False)
TypeError: Run.__init__() got an unexpected keyword argument 'do_exit'
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/__main__.py", line 45, in <module>
    run(configuration, out)
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/main.py", line 38, in run
    judge(pack)
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/judge/core.py", line 187, in judge
    result_status, currently_open_tab = _process_results(
                                        ^^^^^^^^^^^^^^^^^
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/judge/core.py", line 325, in _process_results
    continue_ = evaluate_context_results(
                ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/judge/evaluation.py", line 296, in evaluate_context_results
    missing_stdout = _evaluate_channel(
                     ^^^^^^^^^^^^^^^^^^
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/judge/evaluation.py", line 113, in _evaluate_channel
    evaluation_result = evaluator(output, actual if actual else "")
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/oracles/programmed.py", line 54, in evaluate
    result = evaluate_programmed(
             ^^^^^^^^^^^^^^^^^^^^
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/judge/programmed.py", line 45, in evaluate_programmed
    return _evaluate_python(bundle, evaluator, expected, actual)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/mnt/X7G8DMYVScTX-2wZKYyjFA/judge/tested/judge/programmed.py", line 221, in _evaluate_python
    exec(f"__tested_test__result = {literal_function_call}", global_env)
  File "<string>", line 1, in <module>
  File "<string>", line 23, in evaluate_value
IndexError: string index out of range

Standard Output:

{"command": "start-judgement"}
{"message": "Pylint gecrasht", "command": "append-message"}
{"message": {"description": "Run.__init__() got an unexpected keyword argument 'do_exit'", "format": "code", "permission": "staff"}, "command": "append-message"}
{"title": "Feedback", "command": "start-tab"}
{"command": "start-context"}

https://dodona.be/en/submissions/16041449/

@niknetniko niknetniko added the bug Something isn't working label Nov 24, 2023
@niknetniko
Copy link
Member Author

It seems that the internal error is not related to the pylint crash, e.g. https://dodona.be/en/submissions/16041495/.

It seems we use a deprecated parameter, which was removed in PyLint 3.0. The Docker images were "recently" updated to use pylint 3.0 in dodona-edu/docker-images#301

This was referenced Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant