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

Explainability #330

Closed
gleitz opened this issue Feb 12, 2021 · 8 comments · Fixed by #381
Closed

Explainability #330

gleitz opened this issue Feb 12, 2021 · 8 comments · Fixed by #381
Assignees

Comments

@gleitz
Copy link
Owner

gleitz commented Feb 12, 2021

Would be nice to pass a flag (--explain) and get the way that howdoi chose the answer.

@gmelodie
Copy link
Collaborator

So here's my design idea for that feature (two birds one stone):

  1. Successful search without --explain, nothing new happens
  2. Successful search with --explain: show where the answer came from
  3. Unsuccessful search (with or without --explain): show where the answer was supposed to come from (plus the error?)

This would introduce the new functionality and also fix problems like #331.

@gleitz
Copy link
Owner Author

gleitz commented Feb 17, 2021

Might be interesting to also show counts (hits on search, answers on Stack Overflow, etc)

@gleitz
Copy link
Owner Author

gleitz commented Mar 5, 2021

I think that #344 is most of the way there. We'll need to look at what happens when you search for a query that returns nothing

python -m howdoi.howdoi ksjfksdjfksd fkdsj -x

cc @KimaruThagna

@gleitz
Copy link
Owner Author

gleitz commented Mar 5, 2021

We also want to not show (use --explain to learn why) when you have already passed -x

@atodorov
Copy link

@gleitz do you mind assigning this to MLH fellow @KimaruThagna

@gleitz
Copy link
Owner Author

gleitz commented Mar 26, 2021

Yes, this is very close and I would like @KimaruThagna's help. BTW I for some reason can't use the Assignees section so I just mention you in the comment here.

@gmelodie gmelodie assigned gmelodie and unassigned gmelodie Mar 30, 2021
@gmelodie
Copy link
Collaborator

I think @KimaruThagna needs to have replied to (aka participated in) this issue at some point so we can assign him.

gleitz added a commit that referenced this issue Apr 14, 2021
gleitz added a commit that referenced this issue Apr 14, 2021
gleitz added a commit that referenced this issue Apr 14, 2021
@gleitz
Copy link
Owner Author

gleitz commented Apr 14, 2021

Proposed PR: #381

@gleitz gleitz self-assigned this Apr 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants