Skip to content

Ahodges/documentation/answerdist #2830

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

Jump to bottom
Merged
mhoeber merged 1 commit into from
Mar 31, 2014
Merged

Ahodges/documentation/answerdist #2830

mhoeber merged 1 commit into from
Mar 31, 2014

Conversation

@lamagnifica
Copy link

@lamagnifica lamagnifica commented Mar 5, 2014

@mulby , @brianhw , @mhoeber , @srpearce please take a look if you have a chance

@lamagnifica
Copy link
Author

lamagnifica commented Mar 7, 2014

@mulby @rocha @brianhw @stroilova please review so that we can get the doc to match the release

mhoeber
mhoeber reviewed Mar 7, 2014
View reviewed changes
docs/en_us/course_authors/source/course_grades.rst Outdated
Copy link
Contributor

@mhoeber mhoeber Mar 7, 2014

Choose a reason for hiding this comment

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

maybe need olga's input, but I think the possible effects of changing policy or problems after the course starts could be worse than a communication problem, and that we want to discourage it.

Copy link
Author

@lamagnifica lamagnifica Mar 7, 2014

Choose a reason for hiding this comment

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

@stroilova please weigh in with your opinion. I got this note from the PMs when I wrote the chapter originally.

Copy link
Author

@lamagnifica lamagnifica Mar 7, 2014

Choose a reason for hiding this comment

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

Per @stroilova , this note is ok if "or problem components " is removed. (Problem components should typically NOT be changed as there are numerous ramifications, but changes to the grading policy are ok)

Copy link
Author

@lamagnifica lamagnifica Mar 20, 2014

Choose a reason for hiding this comment

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

Regrading is NOT required after a change to the grading policy per @stroilova , adjust location or rewrite sentence as needed.

@mhoeber
Copy link
Contributor

mhoeber commented Mar 7, 2014

other than inline questions, 👍

srpearce
srpearce reviewed Mar 7, 2014
View reviewed changes
docs/en_us/course_authors/source/course_grades.rst Outdated
Copy link
Contributor

@srpearce srpearce Mar 7, 2014

Choose a reason for hiding this comment

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

Nit: "Is" should be capitalized (since it's a verb).

@lamagnifica
Copy link
Author

lamagnifica commented Mar 13, 2014

Waiting to publish until the release of the Instructor Dashboard feature

@mhoeber
Copy link
Contributor

mhoeber commented Mar 13, 2014

@lamagnifica ping me when ready to rebase

rocha
rocha reviewed Mar 14, 2014
View reviewed changes
docs/en_us/course_authors/source/course_grades.rst Outdated
Copy link
Contributor

@rocha rocha Mar 14, 2014

Choose a reason for hiding this comment

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

It is very likely that we are going to rename the section for that graphic "Score Distribution" to avoid confusion with grades.

@lamagnifica
Copy link
Author

lamagnifica commented Mar 18, 2014

@shnayder please let me know if any info that would be valuable is missing, or if anything here is incorrect!

lamagnifica
lamagnifica reviewed Mar 19, 2014
View reviewed changes
docs/en_us/course_authors/source/course_grades.rst Outdated
Copy link
Author

@lamagnifica lamagnifica Mar 19, 2014

Choose a reason for hiding this comment

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

@mikigoyal @rocha @shnayder

Copy link

@shnayder shnayder Mar 24, 2014

Choose a reason for hiding this comment

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

"If the question does not have accessibility labels defined, this column will be blank"

@lamagnifica
Copy link
Author

lamagnifica commented Mar 24, 2014

How's this look? @shnayder @mulby @brianhw @stroilova @srpearce

srpearce
srpearce reviewed Mar 24, 2014
View reviewed changes
docs/en_us/course_authors/source/course_grades.rst Outdated
Copy link
Contributor

@srpearce srpearce Mar 24, 2014

Choose a reason for hiding this comment

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

Should there be another option below ("either" implies two)?

Copy link
Contributor

@brianhw brianhw Mar 26, 2014

Choose a reason for hiding this comment

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

(There are two options below. : )

@srpearce
Copy link
Contributor

srpearce commented Mar 24, 2014

Reviewing pass done.

brianhw
brianhw reviewed Mar 25, 2014
View reviewed changes
docs/en_us/course_authors/source/course_grades.rst Outdated
Copy link
Contributor

@brianhw brianhw Mar 25, 2014

Choose a reason for hiding this comment

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

s/acheived/achieved/

@rocha
Copy link
Contributor

rocha commented Mar 26, 2014

@lamagnifica we need to make the changes following @brianhw comments, and rebase the branch. Let me know if I can help.

brianhw
brianhw reviewed Mar 26, 2014
View reviewed changes
docs/en_us/course_authors/source/course_grades.rst Outdated
Copy link
Contributor

@brianhw brianhw Mar 26, 2014

Choose a reason for hiding this comment

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

There is no identifier provided for dropdowns. So this is just for checkboxes and multiple choice problems. And blank also for dropdown problems.

@lamagnifica
Copy link
Author

lamagnifica commented Mar 27, 2014

@brianhw please take a look. @carlos thank you I'll be in touch.
@shnayder , @mulby , @stroilova , @srpearce anything further?

brianhw
brianhw reviewed Mar 28, 2014
View reviewed changes
docs/en_us/course_authors/source/course_grades.rst Outdated
Copy link
Contributor

@brianhw brianhw Mar 28, 2014

Choose a reason for hiding this comment

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

This is never blank. Even if a problem has a single part, that part still has an ID.

@brianhw
Copy link
Contributor

brianhw commented Mar 28, 2014

Very close -- just one more correction, and then the answer distribution doc is good to go.

@lamagnifica
Copy link
Author

lamagnifica commented Mar 28, 2014

@brianhw

@brianhw
Copy link
Contributor

brianhw commented Mar 28, 2014

Thanks, LGTM. 👍

@mhoeber
Copy link
Contributor

mhoeber commented Mar 31, 2014

👍

mhoeber added a commit that referenced this pull request Mar 31, 2014
@mhoeber
Merge pull request #2830 from edx/ahodges/documentation/answerdist
9006423 
Ahodges/documentation/answerdist
@mhoeber mhoeber merged commit 9006423 into master Mar 31, 2014
@mhoeber mhoeber deleted the ahodges/documentation/answerdist branch March 31, 2014 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@lamagnifica @mhoeber @srpearce @rocha @brianhw @shnayder @mulby