Closed diaohancai closed 9 months ago
Attention: 29 lines
in your changes are missing coverage. Please review.
Comparison is base (
ff85e34
) 85.03% compared to head (7d6e8ca
) 84.99%. Report is 4 commits behind head on master.:exclamation: Current head 7d6e8ca differs from pull request most recent head 94bd35b. Consider uploading reports for the commit 94bd35b to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
some tiny comments
Thank you for your guidance.
THX~
Thank you for your guidance.
Purpose of the PR
Main Changes
The current random walk algorithm requires 2 additional features.
Add the following parameters:
String weightProperty
. To implement biased random walk. The higher the weight, the higher the probability of walking.Double defaultWeight
. Provide a default value if the weight is null.Double minWeightThreshold
. Truncate when weight is less than the threshold to avoid too small weight.Double maxWeightThreshold
. Truncate when weight exceeds the threshold to avoid overweighting.Double returnFactor
. Controls the probability of re-walk to a previously walked vertex.Double inOutFactor
. Controls whether to walk inward or outward.For more details about
returnFactor
andinOutFactor
, please refer to the paper《node2vec: scalable feature learning for networks》.Verifying these changes
org.apache.hugegraph.computer.algorithm.sampling.RandomWalkTest
Does this PR potentially affect the following parts?
Documentation Status
Doc - TODO
Doc - Done
Doc - No Need