Flag for alternative to ellipsis in solution replacement about otter-grader HOT 7 CLOSED

This is a pretty simple fix... although I think replacing with NULL might give students the wrong idea. What about, e.g., a string like "YOUR CODE HERE" or some other value indicating that the statement needs to be replaced? Or a line with NULL # YOUR CODE HERE as an indicator to students?

from otter-grader.

I agree. For inline prompts I like NULL # YOUR CODE HERE and for multi-line solution removal just

## YOUR CODE HERE

Even "..." would work better in R than a regular ellipsis and matches the python, but I guess this also doesn't really send the right message to students.

from otter-grader.

I have another thought which is specific to RMarkdown that I think is the best case solution. I assume this is much more difficult to implement, however:

For any code chunks with solution removal, in addition to adding the placeholder text, set the chunk option eval=FALSE. This way, R would not run any code without students explicitly setting eval=TRUE, after attempting a solution. With eval=FALSE you could still knit the assignment with unfinished / unattempted solutions, which would be useful for releasing a pdf of the assignment template, for example.

from otter-grader.

While this is do-able, I am hesitant for the same reason as just indicating NULL. I don't think students will realize they need to change this when the knit and so their knitted document won't include the output of those cells. The goal here should be something that is obvious and low-effort for students. I think that NULL # YOUR CODE HERE is the best option.

from otter-grader.

Opened #124 and released v1.1.3 which implements this change.

from otter-grader.

Came to open a similar issue.

I don't think students will realize they need to change this

I think this is the same for ... — thinking # YOUR CODE HERE would be much more explicit. Using Python in my case.) Happy to open a new issue if that's better.

from otter-grader.

@afeld yes, please open a new issue for this feature request.

from otter-grader.