Add per-question LLM/Standard mode toggle for async peer instruction (discussion)#1173
Draft
sethbern wants to merge 7 commits intoRunestoneInteractive:mainfrom
Draft
Add per-question LLM/Standard mode toggle for async peer instruction (discussion)#1173sethbern wants to merge 7 commits intoRunestoneInteractive:mainfrom
sethbern wants to merge 7 commits intoRunestoneInteractive:mainfrom
Conversation
restore studentVoteCount increment for sync PI grading
Member
|
I'm torn, I know @barbarer wants this, but very few people use async PI and to add more compexity to this page is worrisome. I understand it is extra clicks to enable, the async, but we cannot have everything on the front page. I'm thinking about it.... |
Contributor
Author
|
I think it might make more sense and reduce interface clutter to integrate the async option into the Visibility status dropdown rather than having a separate toggle. Instead of a standalone async toggle, what if the Visibility dropdown could include an option like "Visible + Async PI (after X time)" that enables async peer instruction when the visibility conditions are met? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR gives instructors the ability to toggle async peer instruction on and off and choose which PI questions use an LLM. This is what I have so far I would love any feedback, especially with the assignment dashboard since there have been recent changes. @bnmnetp @barbarer
Instructors can toggle async peer mode on/off from the assignment list
Each peer question can be set to LLM or Standard mode individually (defaults to Standard)
LLM option only shows up if there's a valid API key configured
If the API key is broken students get a fallback to standard discussion instead of an error
Deleting an API key flips everything back to Standard and re-adding restores previous settings
Students see a small disclaimer when chatting with the AI peer