SOBotics / Higgs

A generic dashboard for viewing and providing feedback to SOBotics bots.
GNU General Public License v3.0
4 stars 0 forks source link

Skip particular post to give feedback. #63

Closed sri-shree closed 5 years ago

sri-shree commented 5 years ago

Some particular post I am not sure what feedback is appropriate so I want to skip that post. But I can't find any option to skip that post and get new post. So it's useful to allow reviewers to skip that particular post on review queue. https://higgs.sobotics.org/review

rjrudman commented 5 years ago

@sri-shree Currently only heat detector reports are coming through. For these, you should just send the 'sk' feedback

sri-shree commented 5 years ago

sk is not a part of feedback ? I miss understood it ??

rjrudman commented 5 years ago

Can you show me a screenshot of the review item you're doing? There should be 'sk' 'nc' 'fp' 'fn' 'tp'

sri-shree commented 5 years ago

I completely miss under stood about sk. Now it's clear. No re-pro. Thanks and please closed this issue please.

rjrudman commented 5 years ago

@sri-shree 'sk' is actually a feedback type for heat detector, but it's designed to skip the review if the person is unsure :)

sri-shree commented 5 years ago

Yep that's why I am confuse :) . Thanks.

double-fault commented 5 years ago

@rjrudman Would it actually be possible to implement a skip button in Higgs, not as a feedback button? Most bots, unlike HD, do not have a skip feedback.

rjrudman commented 5 years ago

@fortunate-man technically, yep, it's possible. However, Higgs doesn't really have the concept of reviews (yet). The review page simply displays a report which you haven't supplied feedback to yet, and which hasn't met the configured amount of required feedback. If/when other bots integrate with Higgs, I'll look into it.

Holding off on it now because of 'sk', and to see what other bots might require review-wise before implementing it

double-fault commented 5 years ago

@rjrudman ok, thanks!

rjrudman commented 5 years ago

@fortunate-man I've added an issue here: https://github.com/SOBotics/Higgs/issues/64 so it doesn't get lost/forgotten :)