Skip to content

[Feature: Autograding] Adds autograding example for database query gradeables #26

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

Merged
merged 6 commits into from
Dec 13, 2019

Conversation

shwehtom89
Copy link
Contributor

This PR is a copy from the Submitty repo

Please check if the PR fulfills these requirements:

  • The PR title and message follows our guidelines
  • Tests for the changes have been added/updated (if possible)
  • Documentation has been updated/added if relevant

What is the current behavior?

Currently there is no example for instructors to follow to create autograded database assignments in Docker.
There are no logging for database query metrics like (connection time, database ready time, query time)
This PR Is a continuation of #4649 which adds more logging for database assignments.

What is the new behavior?

  • Adds example gradeable configuration.
  • Provides a simple database example in Docker
  • Adds example database 'client' for autograding
  • Logs autograding database metrics (database ready time, query times)
    This file is stored in /var/local/submitty/courses/<semester>/<course>/results/<gradeableid>/<submitter>/<number>/details/<test number>/main/query_metrics.txt

Other information?

In order to build this gradeable, you need the example image.
Inside the vm run,
docker pull goofykubers/example_db_autograding:latest

Special Thanks for Prof Samuel B Johnson for providing a starting point for this PR.

@shwehtom89 shwehtom89 requested a review from emaicus December 11, 2019 17:28
Copy link
Contributor

@emaicus emaicus left a comment

Choose a reason for hiding this comment

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

Tested, works.

@bmcutler bmcutler merged commit 9286e29 into master Dec 13, 2019
@bmcutler bmcutler deleted the example_database branch May 29, 2020 01:34
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