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.

Sign up for GitHub

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

OctaveCodeQuestion: Octave/MATLAB support in RELATE #633

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

OctaveCodeQuestion: Octave/MATLAB support in RELATE #633

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
4358682
Merge pull request #4 from inducer/master
davis68 Dec 12, 2018
c0192c8
Post basic Octave support.
davis68 Dec 14, 2018
50ca0d1
Use two steps to handle copying code over to Docker image.
davis68 Dec 15, 2018
0e8768d
Flatten Docker image back into one.
davis68 Dec 30, 2018
3491cd3
Include Docker image by name. It needs to be built first.
davis68 Dec 30, 2018
6f8980c
First stab at a CodeQuestion interface. UNTESTED but pushed for backup
davis68 Apr 2, 2019
146c0aa
Fixed the CodeQuestion blocks.
davis68 Apr 3, 2019
3700301
Everything seems to be working properly now.
davis68 Apr 3, 2019
ef05a70
Post davis68 version of docker image.
davis68 Apr 3, 2019
8bfe887
Post changes to tests.
davis68 Apr 3, 2019
a4cedef
Post changes to tests.
davis68 Apr 3, 2019
1e6051b
Post tests.
davis68 Apr 9, 2019
cd7cfca
Fix some imports from older version of code.
davis68 Apr 9, 2019
c477cd0
Fix some imports from older version of code.
davis68 Apr 9, 2019
8c2d4f0
Turn off debugging.
davis68 Apr 11, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,3 +28,6 @@ git-roots

/.idea
/.env

my_todo
password
7 changes: 4 additions & 3 deletions bin/relate
100644 → 100755
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,8 @@ def expand_yaml(yml_file, repo_root):
def test_code_question(page_desc, repo_root):
if page_desc.type not in [
"PythonCodeQuestion",
"PythonCodeQuestionWithHumanTextFeedback"]:
"PythonCodeQuestionWithHumanTextFeedback",
"OctaveCodeQuestion"]:
return

print(75*"-")
Expand All @@ -64,11 +65,11 @@ def test_code_question(page_desc, repo_root):

correct_code = getattr(page_desc, "correct_code", "")

from course.page.code_runpy_backend import \
from course.page.code_run_backend import \
substitute_correct_code_into_test_code
test_code = substitute_correct_code_into_test_code(test_code, correct_code)

from course.page.code_runpy_backend import run_code, package_exception
from course.page.code_run_backend import run_code, package_exception

data_files = {}

Expand Down
2 changes: 2 additions & 0 deletions course/page/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,8 @@
ChoiceQuestion, MultipleChoiceQuestion, SurveyChoiceQuestion)
from course.page.code import (
PythonCodeQuestion, PythonCodeQuestionWithHumanTextFeedback)
from course.page.code import (
OctaveCodeQuestion)
from course.page.upload import FileUploadQuestion

__all__ = (
Expand Down
6 changes: 6 additions & 0 deletions course/page/base.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -768,6 +768,12 @@ def __init__(self, vctx, location, page_desc):
if title is None:
try:
md_body = self.markup_body_for_title()
except AttributeError:
#TODO XXX
from warnings import warn
warn(_("PageBaseWithTitle subclass '%s' does not implement "
"markup_body_for_title()")
% type(self).__name__)
except NotImplementedError:
from warnings import warn
warn(_("PageBaseWithTitle subclass '%s' does not implement "
Expand Down
Loading