-
Notifications
You must be signed in to change notification settings - Fork 2
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
Some ideas for extra heuristics #29
Comments
@pombredanne The second case is definitely better off being as a scancode-toolkit change IMHO. The first one could be added as a results-analyzer heuristic to detect false-positives, as there could be short references after 100 lines more often? Will integrate this as soon as the plugin is ready. These extra heuristics would be very easy to add once the structure is ready, and would be very important in the analysis process, we could also look at more statistics to get more of these! I'm pushing a PR for the docs at #22 where the current classification/heuristics are detailed. Also, in both cases, shouldn't they be Thanks! |
Adds an extra heuristic for false positives detection, based on line number and rule length, and their related tests. Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
This commit adds the first case of the extra heuristic, and partially solves this issue - #34 |
Adds an extra heuristic for false positives detection, based on line number and rule length, and their related tests. Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
Adds an extra heuristic for false positives detection, based on line number and rule length, and their related tests. Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
Adds an extra heuristic for false positives detection, based on line number and rule length, and their related tests. Fixes error reporting issue and other keyerrors. Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
Adds an extra heuristic for false positives detection, based on line number and rule length, and their related tests. Fixes error reporting issue and other keyerrors. Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
Adds an extra heuristic for false positives detection, based on line number and rule length, and their related tests. Fixes error reporting issue and other keyerrors. Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
Adds an extra heuristic for false positives detection, based on line number and rule length, and their related tests. Fixes error reporting issue and other keyerrors. Signed-off-by: Ayan Sinha Mahapatra <[email protected]>
from a chat with @maxhbr
a short license reference (single word?) detected after several 100 lines (say after the 1000's line is likely false positive
a short license reference (single word?) detected in a binary and where the actual text has a different case than the case of the rule is likely false positive. For instance
gPL
orgPl
. This could be also things that are done directly in Scancode toolkitThe text was updated successfully, but these errors were encountered: