{"action":"closed","issue":{"url":"https://api.github.com/repos/gmum/gmum.r/issues/489","labels_url":"https://api.github.com/repos/gmum/gmum.r/issues/489/labels{/name}","comments_url":"https://api.github.com/repos/gmum/gmum.r/issues/489/comments","events_url":"https://api.github.com/repos/gmum/gmum.r/issues/489/events","html_url":"https://github.com/gmum/gmum.r/issues/489","id":77065154,"number":489,"title":"Another round of tests [mostly SVM]","user":{"login":"lejlot","id":1747438,"avatar_url":"https://avatars.githubusercontent.com/u/1747438?v=3","gravatar_id":"","url":"https://api.github.com/users/lejlot","html_url":"https://github.com/lejlot","followers_url":"https://api.github.com/users/lejlot/followers","following_url":"https://api.github.com/users/lejlot/following{/other_user}","gists_url":"https://api.github.com/users/lejlot/gists{/gist_id}","starred_url":"https://api.github.com/users/lejlot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lejlot/subscriptions","organizations_url":"https://api.github.com/users/lejlot/orgs","repos_url":"https://api.github.com/users/lejlot/repos","events_url":"https://api.github.com/users/lejlot/events{/privacy}","received_events_url":"https://api.github.com/users/lejlot/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/gmum/gmum.r/labels/bug","name":"bug","color":"e11d21"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-16T15:38:50Z","updated_at":"2015-08-22T23:29:13Z","closed_at":"2015-08-22T23:29:13Z","body":"[DEV branch]\r\n\r\nMajor\r\n\r\n0. [GENERAL] Great number of warnings during compilation - we should fix it in order to be accepted by CRAN\r\n1. [GENERAL] What is \"x\"? and \"y\" .. and \"x.cec\" and \"y.cec\"\r\n\r\n        > x\r\n        standardGeneric for \"x\" defined from package \"gmum.r\"\r\n\r\n        function (c) \r\n        standardGeneric(\"x\")\r\n        <environment: 0x83a88e0>\r\n        Methods may be defined for arguments: c\r\n        Use  showMethods(\"x\")  for currently available ones.\r\n\r\n  in general, we have horribly \"clutered\" namespace, compare what one sees after typing gmum.r::<tab><tab> and e1071::<tab><tab>\r\n\r\n2. [SVM] Transductive SVM fails with multiclass (it is fine, but we should at least produce a nice error message)\r\n3. [SVM] Plotting of Transductive SVM sometimes fails (iris is iris dataset loaded with data)\r\n\r\n        > y = c(rep(\"a\", 100), rep(\"b\", 10), rep(0, 40))\r\n        > svm.transduction <- SVM(iris[,-5], y, transductive.learning=TRUE, core=\"svmlight\")\r\n        Komunikat ostrzegawczy:\r\n        In SVM.default(iris[, -5], y, transductive.learning = TRUE, core = \"svmlight\") :\r\n          It is recommended to pass y as factor\r\n        > plot(svm.transduction)\r\n        Błąd w `levels<-.factor`(`*tmp*`, value = c(\"a\", \"b\")) : \r\n          number of levels differs\r\n        Dodatkowo: Komunikat ostrzegawczy:\r\n        In .local(x, ...) :\r\n          Only 2 dimension plotting is supported for multiclass. Plotting using cols parameter\r\n\r\n\r\n\r\nMinor\r\n\r\n1. [SVM] Preprocessing type is absent in \"summary\" when using multiclass\r\n2. [SVM] In multiclass scenario model$models[[1]]$call has horribly long output (all variables are raw!)\r\n3. [SVM] What is $pick in multiclass svm?\r\n4. [SVM] Methods names are inconsistent between multiclass svm and single svm (kernel vs getKernel, etc.)\r\n5. [SVM] Summary does not include C parameter, nor gamma\r\n6. [SVM] SVM accepts gamma=0 (it should not)\r\n7. [SVM] While provifing bad C creates nice error message, providing bad gamma outputs simply \"bad SVM parameters\", similarly putting C=\"a\" outputs \"not comatible with requested type\"\r\n8. [SVM] There are ugly messages when one tries to getW() on kernel SVM (W is not \"decision boundary\", by a hyperplane normal)\r\n\r\n        ERROR: Decision boundary is not available with non-linear kernel\r\n        BŁĄD: basic_string::_S_construct null not valid\r\n\r\n9. [SVM] Transduction SVM has no iterations? getIterations() returns -1"}}