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

[Feature Request] i need 0('N')-1('Y') with one bubble... is it possible? #61

Closed
ibrahimkilic opened this issue Oct 3, 2022 · 6 comments · Fixed by #91
Closed

[Feature Request] i need 0('N')-1('Y') with one bubble... is it possible? #61

ibrahimkilic opened this issue Oct 3, 2022 · 6 comments · Fixed by #91

Comments

@ibrahimkilic
Copy link

ibrahimkilic commented Oct 3, 2022

I want 'yes' when bubble is full and 'no' when empty, in result csv file.... is it posible?
example

@Udayraj123 Udayraj123 changed the title i need 0('N')-1('Y') with one bubble... is it possible? [Feature Request] i need 0('N')-1('Y') with one bubble... is it possible? Oct 4, 2022
@Udayraj123
Copy link
Owner

@ibrahimkilic We can introduce a new config in template.json for defining what to use for empty value.
I have asked in the community for anyone who can pick it up. We can discuss what should the schema be for most generalised version.

p.s. previously there was a variable used for this. Had to remove it because it would cause some issues in double-integer type questions.

@Udayraj123
Copy link
Owner

For now I can suggest a small hack in the code here.

Add an else block there and put this(with correct indentation)

else:
    if (q in ["q1", "q2", "q3", "q4", "q5"]):
        omr_response[q] = "no"

@sanmay321
Copy link

@ibrahimkilic hey I would like to work on this issue assign me, please.

@Udayraj123
Copy link
Owner

@sanmay321 we're following a process of discussing on discord before getting assigned. Please start a discussion over there :)

@Udayraj123
Copy link
Owner

let's discuss how to generalize

@ibrahimkilic
Copy link
Author

Thanks, bro...

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

Successfully merging a pull request may close this issue.

3 participants