Skip to content

Commit

Permalink
Merge pull request #3654 from lkuchenb/fix/ModelSelectionParameters_s…
Browse files Browse the repository at this point in the history
…et_values

Fix CModelSelectionParameters::set_values()
  • Loading branch information
karlnapf committed Feb 23, 2017
2 parents ce65285 + 251c7d4 commit 9546458
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 12 deletions.
11 changes: 0 additions & 11 deletions src/shogun/modelselection/ModelSelectionParameters.cpp
Expand Up @@ -92,17 +92,6 @@ void CModelSelectionParameters::append_child(CModelSelectionParameters* child)
m_child_nodes->append_element(child);
}

template <class T>
void CModelSelectionParameters::set_values(const SGVector<T>& values,
EMSParamType value_type)
{
/* possibly delete old range values */
delete_values();
m_values=values.vector;
m_values_length=values.vlen;
m_value_type=value_type;
}

void CModelSelectionParameters::build_values(float64_t min, float64_t max,
ERangeType type, float64_t step, float64_t type_base)
{
Expand Down
9 changes: 8 additions & 1 deletion src/shogun/modelselection/ModelSelectionParameters.h
Expand Up @@ -104,7 +104,14 @@ class CModelSelectionParameters: public CSGObject
* @param value_type type of the provided vector
*/
template <class T>
void set_values(const SGVector<T>& values, EMSParamType value_type);
void set_values(const SGVector<T>& values, EMSParamType value_type)
{
/* possibly delete old range values */
delete_values();
m_values=values.vector;
m_values_length=values.vlen;
m_value_type=value_type;
}

/** SG_PRINT's the tree of which this node is the base
*
Expand Down

0 comments on commit 9546458

Please sign in to comment.