Encoder-decoder task support

sio2project / sioworkers

GNU General Public License v3.0

11 stars 22 forks source link

Encoder-decoder task support #16

Open top-sekret opened 1 year ago

top-sekret commented 1 year ago

This is a draft pull request that adds support for encoder-decoder tasks, as seen at latest OI finals.

Notes:

Currently, the decoder could get the encoder input just by opening a file with the correct name. This will be addressed.
Lacks unit tests.

Also see sio2project/oioioi#191.

top-sekret commented 1 year ago

I have addressed the security issue and rebased this branch. I now mark it as ready for review, as we may come up with unit tests during the review process.

top-sekret commented 1 year ago

Also note that this pull request can be merged before the OIOIOI one, because it does not alter any visible behavior. The converse is of course not true.

top-sekret commented 1 year ago

I really wish this could be made to work. However, I am being stopped by what I think is a sio2jail bug.

MasloMaslane commented 1 year ago

I think it's because Github runners don't give access to perf counters. But recently we created our own runner that can use sio2jail.

top-sekret commented 1 year ago

I think it's because Github runners don't give access to perf counters. But recently we created our own runner that can use sio2jail.

I mean they won't even run locally.

top-sekret commented 11 months ago

Okay, I'm out of patience. The tests do work. However, the CI is absolute crap and can't even set up Python.

I am reverting (actually rebasing out to get rid of the mess) all CI related commits. I will be running the tests on docker.io/library/python:3.9 and docker.io/library/debian:bookworm manually to ensure that the changes work.