Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Improve the implied technologies management and confidence #41

Open
tristanlatr opened this issue Nov 19, 2020 · 0 comments
Open

Improve the implied technologies management and confidence #41

tristanlatr opened this issue Nov 19, 2020 · 0 comments

Comments

@tristanlatr
Copy link
Collaborator

Since #36, implied technologies that contain the confidence are parsed and if it's equal or greater than 50, it's added to the results. But we do not track this confidence afterwards.
It would be good to have some control about this behaviour in the analyze method arguments.

Additionally, this feature is only working when using the analyze method, not other.

Also, I'm not too sure what get_confidence(app_name) is doing, it's like it returns the sum of confidences got from the technologies.json file regex expressions, for a specified app. But it does not returns the confidence that a certain website has an app. And I think it's buggy since it always returns a empty list to me.

The confidence per technology is a missing feature from this implementation yet

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant