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

Include Harvested data in suggestions #385

Open
jeffmcaffer opened this issue Feb 7, 2019 · 0 comments
Open

Include Harvested data in suggestions #385

jeffmcaffer opened this issue Feb 7, 2019 · 0 comments

Comments

@jeffmcaffer
Copy link
Member

jeffmcaffer commented Feb 7, 2019

There are cases where summarization excludes some harvested data, for example, the license score min-bar is 80 for ScanCode. While this makes sense for automated cases, if the user is looking at fixes, it would be good for them to see that, for example, one of the tools was 75% sure there was an Apache license. See #383 for an example of a related scenario.

So the suggestion (pun!) here is for "getSuggestions" to recompute the definition but with fuzzier settings. Then anything that is different becomes a suggestion. This should actually be pretty simple, just would require passing the fuzzy factors into the summarizer.

@storrisi there might be some changes for how the UI presents suggestions. It likely should be general as in the future there will be many different "sources" of suggestion.

@storrisi storrisi added this to the February 2019 milestone Feb 11, 2019
@storrisi storrisi modified the milestones: February 2019, March 2019 Feb 28, 2019
@jeffmendoza jeffmendoza removed this from the March 2019 milestone Nov 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants