WMFP metric #293
Closed
WMFP metric #293
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
+ Added new metric wmfp with subjective coefficient 0.65 + Added test for the metric
+ Updated return types for nested functions + Updated types of values inside `statements` and `variables` parameters
+ added return annotations for the 2 subfunctions
|
I apologize for the premature timing PR, when running locally I didn’t notice any problems with the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@yegor256 I introduce the new metric you mentioned here #26
My code contains 2 subfunctions for calculating some additional parameters and one main function that groups everything together and counts the final result.
Right now the function analyzes the following things in the code:
Only these features are considered because I found them in test case.
Note: The main function has a hyperparameter
0.65incalculating complexity adjustment factorstage. In the description it's said we can use some parameters like this one for manipulating the total count. This number exactly is taken randomly, thus, maybe it's needed to be changed later or even removed.