Skip to content
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.

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

Show all combinations #60

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Show all combinations #60

wants to merge 2 commits into from

Conversation

lukebaker
Copy link
Member

Related: allourideas/pairwise-api#43

This pull request adds the ability for admins (not wiki survey creators) to select which prompt selection algorithm to use. I figured you weren't keen on having wiki survey creators modify which prompt selection to use, so I limited it to only admins. The default is no prompt selection algorithm is sent to pairwise and pairwise decides which algorithm to use. Prompt selection algorithm options include "Catchup" and "All combos".

@msalganik does this look good to you?

This prompt_algorithm value will be passed along to pairwise when a new
prompt is required (voting, skipping, etc.). When not set, pairwise will
decide which algorithm is used. This does require some coordination
between pairwise and AOI to determine what algorithms are supported.
@msalganik
Copy link
Member

Sounds good. Yes, I think we want to role this out slowly. In my other message I asked about having a blog post that we can link to. I think that should be part of the process of rolling out this new feature.

@bazooka720
Copy link

Curious @msalganik @lukebaker...was this resolved (i mean tested)? I see that it was not merged to master?

@msalganik
Copy link
Member

@bazooka720 we have not been doing active development of allourideas for some time. I would not expect this to be added to master for some time. Sorry.

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.

3 participants