Skip to content

Remove submitty_count from csci1200#41

Open
dagemcn wants to merge 4 commits into
mainfrom
deprecate_submitty_count_DI_RPI
Open

Remove submitty_count from csci1200#41
dagemcn wants to merge 4 commits into
mainfrom
deprecate_submitty_count_DI_RPI

Conversation

@dagemcn

@dagemcn dagemcn commented Jun 25, 2026

Copy link
Copy Markdown
Member

What is the current behavior?

Fixes Submitty Issue#11784 - This one of the companion PRs to Submitty PR#12609.

What is the new behavior?

submitty_count is longer installed in preparation for deprecation.

Other information?

This is a breaking change. Any autograding that uses the old submitty_count instead of submitty_count_ts will no longer work.

  1. Follow the instructions on this page up to step 6 with the Dockerfile modified in this PR. (you may be able to reuse the one I used dagemcn/dockerimages:csci1200-test adding it through docker UI)
  2. After you have added the new docker image with the DockerUI, update the config of any gradeable with submitty_count with
"autograding_method": "docker",
"container_options": {
    "container_image": "IMAGE_NAME"
},

where IMAGE_NAME is the name of the image, for example, dagemcn/dockerimages:csci1200-test if you reused mine.
3. Create a gradeable using submitty_count, (Tutorial examples 04, 06, and 07 all utilize submitty_count)
4. Test that submitty_count will not work in the config and that submitty_count_ts successfully runs
5. Also test that regular autograding works successfully on the image

Attached is a zip of a sample solution and gradeable config with both submitty_count and submitty_count_ts commands present for testing.
example.zip

@roye2 roye2 self-requested a review July 6, 2026 20:09

@roye2 roye2 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had a fair amount of trouble with the setup for reviewing this PR. Eventually I got to this point, where I uploaded a configuration using submitty_count_ts and got this as the autograding result:

Image

In the docker container, I am able to run the submitty_count_ts commands afaik.

Here is what my config that I uploaded looks like:

Image

Edit, I just realized I forgot to put the -l after submitty_count_ts. I will test again shortly

@roye2 roye2 self-requested a review July 7, 2026 14:04

@roye2 roye2 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After correcting my setup, I was able to verify that a) submitty_count no longer works on this docker image and b) submitty_count_ts does work and autograding works as normal. Approving.

@bmcutler bmcutler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's call this tag summer26

@dagemcn dagemcn marked this pull request as ready for review July 9, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants