kwargs ~works~ for objects ~already~

examples/meta/src/meta_api/kwargs.sg

@@ -22,17 +22,14 @@
 # This also fails (keywords not allowed outside initialisation variables)
 #kernel_factory("GaussianKernel", a=glob_fun(ordinary_argument))
 
+# doesnt work in ruby (TODO, don't allow in meta grammar)
+# KernelMachine svm2 = kernel_machine("LibSVM", kernel=kernel("GaussianKernel"))
+
+
 
 # Real example
 # ------------
 
-# The following lines fail in Octave because of a float literal issue
-#KernelMachine svm = kernel_machine("LibSVM", C1=2.0)
-#GaussianKernel k(log_width=2.0)
-#k.put("log_width", 4.0)
-
-# The following line fails in Python because of a bool literal issue
-#GaussianKernel k2(log_width=14.0, lhs_equals_rhs=True)
+Kernel k = kernel("GaussianKernel", log_width=2.0)
+KernelMachine svm = kernel_machine("LibSVM", C1=1.0, kernel=k)
 
-# The following line currently gives a type error
-#svm.put("kernel", k)

src/interfaces/swig/SGBase.i

@@ -508,22 +508,3 @@ copy_reg._reconstructor=_sg_reconstructor
 %}
 
 #endif /* SWIGPYTHON  */
-
-%include <shogun/lib/basetag.h>
-%include <shogun/lib/tag.h>
-%include <shogun/base/SGObject.h>
-
-%define SUPPORT_TAG(camel_type, short_type, type)
-    %template(Tag ## camel_type) shogun::Tag<type>;
-    %template(put) shogun::CSGObject::put<type>;
-    %template(put) shogun::CSGObject::put<type, void>;
-    %template(put) shogun::CSGObject::put<type>;
-    %template(get_ ## short_type) shogun::CSGObject::get<type, void>;
-    %template(has) shogun::CSGObject::has<type>;
-    %template(has_ ## short_type) shogun::CSGObject::has<type, void>;
-%enddef
-
-SUPPORT_TAG(String, string, std::string)
-SUPPORT_TAG(Float64, float, float64_t)
-SUPPORT_TAG(Int64, int, int64_t)
-SUPPORT_TAG(Object, object, CSGObject*)

src/shogun/base/SGObject.cpp

@@ -1012,3 +1012,40 @@ CSGObject* CSGObject::create_empty() const
 	SG_REF(object);
 	return object;
 }
+
+namespace shogun
+{
+#define SGOBJECT_PUT_DEFINE(T) \
+void CSGObject::put(const std::string& name, T const & value)  throw(ShogunException)\
+{ \
+	Tag<T> tag(name); \
+	put(tag, value); \
+}
+
+SGOBJECT_PUT_DEFINE(SGVector<int32_t>)
+SGOBJECT_PUT_DEFINE(SGVector<float64_t>)
+SGOBJECT_PUT_DEFINE(CSGObject*)
+
+#define PUT_DEFINE_CHECK_AND_CAST(T) \
+else if (has(Tag<T>(name))) \
+	put(Tag<T>(name), (T)value);
+
+#define SGOBJECT_PUT_DEFINE_NUMBER(numeric_t) \
+void CSGObject::put(const std::string& name, numeric_t const & value)  throw(ShogunException)\
+{ \
+	/* use correct type of possible, otherwise cast-convert */ \
+	if (has(Tag<numeric_t>(name))) \
+		put(Tag<numeric_t>(name), value); \
+	PUT_DEFINE_CHECK_AND_CAST(int32_t) \
+	PUT_DEFINE_CHECK_AND_CAST(float32_t) \
+	PUT_DEFINE_CHECK_AND_CAST(float64_t) \
+	else \
+		/* if nothing works, moan about original type */ \
+		put(Tag<numeric_t>(name), value); \
+}
+
+SGOBJECT_PUT_DEFINE_NUMBER(int32_t)
+SGOBJECT_PUT_DEFINE_NUMBER(float32_t)
+SGOBJECT_PUT_DEFINE_NUMBER(float64_t)
+
+};

src/shogun/base/SGObject.h

@@ -364,18 +364,22 @@ class CSGObject
 		}
 	}
 
-	/** Setter for a class parameter, identified by a name.
-	 * Throws an exception if the class does not have such a parameter.
-	 *
-	 * @param name name of the parameter
-	 * @param value value of the parameter along with type information
-	 */
-	template <typename T, typename U = void>
-	void put(const std::string& name, const T& value) throw(ShogunException)
-	{
-		Tag<T> tag(name);
-		put(tag, value);
-	}
+
+#define SGOBJECT_PUT_DECLARE(T) \
+	/** Setter for a class parameter, identified by a name. \
+	 * Throws an exception if the class does not have such a parameter. \
+	 * \
+	 * @param name name of the parameter \
+	 * @param value value of the parameter along with type information \
+	 */ \
+	void put(const std::string& name, T const & value) throw(ShogunException);
+
+	SGOBJECT_PUT_DECLARE(int32_t)
+	SGOBJECT_PUT_DECLARE(float32_t)
+	SGOBJECT_PUT_DECLARE(float64_t)
+	SGOBJECT_PUT_DECLARE(SGVector<int32_t>)
+	SGOBJECT_PUT_DECLARE(SGVector<float64_t>)
+	SGOBJECT_PUT_DECLARE(CSGObject*)
 
 	/** Getter for a class parameter, identified by a Tag.
 	 * Throws an exception if the class does not have such a parameter.