Skip to content

Commit

Permalink
fix lhs / rhs mixup in free_feature_vector (how did this ever work?!)
Browse files Browse the repository at this point in the history
  • Loading branch information
Soeren Sonnenburg committed Oct 13, 2011
1 parent 1d5d860 commit 74bf297
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/shogun/kernel/WeightedDegreePositionStringKernel.cpp
Expand Up @@ -763,7 +763,7 @@ float64_t CWeightedDegreePositionStringKernel::compute_by_tree(int32_t idx)
for (int32_t i=0; i<len; i++)
vec[i]=alphabet->remap_to_bin(char_vec[i]);

((CStringFeatures<char>*) lhs)->free_feature_vector(char_vec, idx, free_vec);
((CStringFeatures<char>*) rhs)->free_feature_vector(char_vec, idx, free_vec);

for (int32_t i=0; i<len; i++)
sum += tries.compute_by_tree_helper(vec, len, i, i, i, weights, (length!=0)) ;
Expand Down Expand Up @@ -802,7 +802,7 @@ void CWeightedDegreePositionStringKernel::compute_by_tree(
for (int32_t i=0; i<len; i++)
vec[i]=alphabet->remap_to_bin(char_vec[i]);

((CStringFeatures<char>*) lhs)->free_feature_vector(char_vec, idx, free_vec);
((CStringFeatures<char>*) rhs)->free_feature_vector(char_vec, idx, free_vec);

for (int32_t i=0; i<len; i++)
{
Expand Down

0 comments on commit 74bf297

Please sign in to comment.