Skip to content

Add per-question LLM/Standard mode toggle for async peer instruction (discussion)#1173

Draft
sethbern wants to merge 7 commits intoRunestoneInteractive:mainfrom
sethbern:async-toggle
Draft

Add per-question LLM/Standard mode toggle for async peer instruction (discussion)#1173
sethbern wants to merge 7 commits intoRunestoneInteractive:mainfrom
sethbern:async-toggle

Conversation

@sethbern
Copy link
Contributor

@sethbern sethbern commented Mar 8, 2026

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

    • image
  • Each peer question can be set to LLM or Standard mode individually (defaults to Standard)

    • LLM option only appears when async mode is enabled
    • Edit All lets you bulk switch all questions at once
    • image
    • image
  • LLM option only shows up if there's a valid API key configured

    • image
  • If the API key is broken students get a fallback to standard discussion instead of an error

    • this also works if somebody is in the middle of a conversation with the llm
    • image
  • 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

@bnmnetp
Copy link
Member

bnmnetp commented Mar 10, 2026

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....

@sethbern
Copy link
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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants