mitodl / mitx-grading-library

The MITx Grading Library, a python grading library for edX
https://edge.edx.org/courses/course-v1:MITx+grading-library+examples/
BSD 3-Clause "New" or "Revised" License
14 stars 9 forks source link

Version 1.2 documentation #106

Closed ChristopherChudzicki closed 6 years ago

ChristopherChudzicki commented 6 years ago

Some of these are very close to done.

My version of the documentation website My version of demo course

Note: I changed the theme to mkdocs-cinder, because I wanted one level deeper on the individual page table of contents. Could explore other themes, but not going to worry about it now.

ChristopherChudzicki commented 6 years ago

@jolyonb This is going to be a pretty big PR. I updated the to-do list at the top. No rush at all, but if you want to look at any of it, I consider what's been pushed to be ready-for-review.

My goal will be to finish some of the course demos for comparer_functions before tomorrow's meetings. (But I probably won't write the website documention for it before the meetings. )

jolyonb commented 6 years ago

I'm sure this one is going to be huge -- we're catching up on two months' worth of features! I'm going to hold off on looking/making comments until you're done, I think.

I agree that focusing on some course demos for tomorrow's meeting(s?) is probably a good idea.

jolyonb commented 6 years ago

Can we make a standalone module of comparison functions, rather than having them scattered about randomly?

ChristopherChudzicki commented 6 years ago

Can we make a standalone module of comparison functions, rather than having them scattered about randomly?

Sure! This was my original inclination, though MatrixGrader comparer_functions get a few extra utils.

jolyonb commented 6 years ago

I much prefer a standalone module. I think we should be able to figure out how to make something that works with the appropriate utils ;-) It can be exposed in a comparers module.

ChristopherChudzicki commented 6 years ago

@jolyonb Ready for review. My version of the demo course and website are update, see PR summary.

ChristopherChudzicki commented 6 years ago

rebased to fix merge conflicts

jolyonb commented 6 years ago

Thanks. I'm going to try to do the programmatic part of this tomorrow morning.

jolyonb commented 6 years ago

I'm merging this now, and will make updates in a future PR.