张泊明518370910136 bomingzh
张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-23 20:43:59 +08:00
fix(diff): bugs on diff stdin and numerics

leave it here now. we have not reused the logic.

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-23 20:42:10 +08:00
fix(diff): bugs on diff stdin and numerics

if not testcases_path.with_suffix(".out").exists():

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-23 20:30:58 +08:00
fix(diff): bugs on diff stdin and numerics

No. Just do not append it to the return value and log a warning.

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-23 17:06:10 +08:00
fix(diff): bugs on diff stdin and numerics

If case*.out will always be used in diff parser, we want to ensure it exists to form a valid case.

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-23 08:46:32 +08:00
fix(diff): bugs on diff stdin and numerics

Please check if the current solution works.

张泊明518370910136 pushed to fix/diff at JOJ/JOJ3-config-generator 2025-05-23 08:46:14 +08:00
c3f2b21732 fix: default limit
张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-23 08:10:29 +08:00
fix(diff): bugs on diff stdin and numerics

better check if the *.out file exists in get_testcases

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-23 08:07:15 +08:00
fix(diff): bugs on diff stdin and numerics

We need another pydantic model for auto detected cases. Fields in these cases can be none, which means they are not set and should use with_.default values.

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-23 07:45:05 +08:00
fix(diff): bugs on diff stdin and numerics

Should be applied to other locations in transformers/task.py

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-23 07:44:02 +08:00
fix(diff): bugs on diff stdin and numerics

Which test case will show this problem?

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-22 16:06:21 +08:00
fix(diff): bugs on diff stdin and numerics

What if the with_.default.cpu_limit is not the same as DEFAULT_CPU_LIMIT? Why do we need to set these fields to none?

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-22 16:03:39 +08:00
fix(diff): bugs on diff stdin and numerics

Should be removed. It should be fixed from JOJ3 side. Also in InputFile = Union[LocalFile, MemoryFile, PreparedFile, Symlink, None]

张泊明518370910136 commented on issue JOJ/JOJ3-config-generator#15 2025-05-22 16:02:05 +08:00
Bug in judge/diff parser.

Could you check if problem 1 persists in the latest JOJ3 (after runner-images is built)?

张泊明518370910136 pushed to master at JOJ/JOJ3 2025-05-22 16:00:12 +08:00
5dec3b1bda fix(cmd/joj3): file fields in generate stages
张泊明518370910136 commented on issue JOJ/JOJ3-config-generator#15 2025-05-22 15:44:16 +08:00
Bug in judge/diff parser.

:-( problem 1 is caused by joj3, fixing

张泊明518370910136 reopened issue JOJ/JOJ3-config-generator#15 2025-05-22 15:43:58 +08:00
Bug in judge/diff parser.
张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-22 15:21:49 +08:00
fix(diff): bugs on diff stdin and numerics

good idea, maybe we can also have DEFAULT_CLOCK_LIMIT_MULTIPLIER

张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-22 15:18:27 +08:00
fix(diff): bugs on diff stdin and numerics

why not

class ParserDiff(BaseModel):
    output: Outputs = Outputs()
    default_score: int = DEFAULT_CASE_SCORE
张泊明518370910136 commented on pull request JOJ/JOJ3-config-generator#16 2025-05-22 15:12:26 +08:00
fix(diff): bugs on diff stdin and numerics

why is it removed?

张泊明518370910136 pushed to master at JOJ/JOJ3-config-generator 2025-05-22 01:34:03 +08:00
6375a37069 fix(diff): bugs in auto detect cases (#14)