domjudge icon indicating copy to clipboard operation
domjudge copied to clipboard

Consider hiding sample download for interactive problems

Open nickygerritsen opened this issue 4 years ago • 4 comments

As mentioned in #920 and also discussed during the CCS Specs standards call, for interactive problems the sample data is mostly useful for the validator and not for teams. The sample data mostly contains information for the validator to run.

I believe it makes more sense to hide the download of the samples for interactive problems for this reason. #920 makes it that we can have attachments if one wants multiple data to pass to some testing tool.

Any thoughts?

nickygerritsen avatar Nov 16 '20 20:11 nickygerritsen

Is it just me or are you overloading the concept of 'sample data' if you're adding sample data that's not sample data?

thijskh avatar Nov 17 '20 09:11 thijskh

Yes. But the problem is that they are sample testcases. We do want to have the option to show sample runs to teams. It should show information there.

nickygerritsen avatar Nov 17 '20 09:11 nickygerritsen

I think the problem is that the concept of sample data really splits into two different concepts for interactive problems:

  1. A sample interaction between submission and jury program.
  2. Data required for a provided interaction tool to run your code against locally.

eldering avatar Nov 17 '20 10:11 eldering

I agree. But it seemed we were getting nowhere during the standards call on deciding anything on this. We can decide to just leave it as is now for the time being until the standard gets updated (if ever), but it will give confusion to teams for interactive problems.

nickygerritsen avatar Nov 17 '20 10:11 nickygerritsen

@jsannemo said that there will be a problem spec change.

meisterT avatar Nov 10 '22 08:11 meisterT

see https://github.com/Kattis/problem-package-format/pull/22

meisterT avatar Nov 10 '22 08:11 meisterT

@eldering will take a look at that, afterwards @nickygerritsen can implement this.

nickygerritsen avatar Nov 10 '22 08:11 nickygerritsen