Skip to content
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

Scala grader #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Scala grader #63

wants to merge 2 commits into from

Conversation

DanG100
Copy link
Contributor

@DanG100 DanG100 commented Jun 3, 2019

Some progress towards replace the grader with pure scala. Currently it is only works with cpu tests, I have to think of a way to get it for both unit tests and cpu tests.

This is the output of the example grader which runs a few of the tests.

{ "tests":[
  {     
      "score": 10.0,
      "max_score": 10.0,
      "name": "Single Cycle Add",
      "output": "Passed add1\n" 
  },
  {     
      "score": 10.0,
      "max_score": 10.0, 
      "name": "R-type single cycle", 
      "output": "Passed add1\nPassed add2\nPassed add0\nPassed or\nPassed sub\nPassed and\nPassed xor\nPassed slt\nPassed slt1\nPassed sltu\nPassed sltu1\nPassed sll\nPassed srl\nPassed sra\n"
  },
  {     
      "score": 10.0, 
      "max_score": 10.0,
      "name": "R-type multicycle",
      "output": "Passed addfwd\nPassed swapxor\nPassed power2-512\nPassed power2-1234\nPassed power2--65536\nPassed oppsign-true\nPassed oppsign-false\nPassed rotR\n"
  }
]}

@powerjg
Copy link
Contributor

powerjg commented Jun 3, 2019

Awesome! We don't have to keep the same scalatest syntax for the gradescope version of the unit tests, btw.

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.

2 participants