{"action":"created","issue":{"url":"https://api.github.com/repos/flowersteam/explauto/issues/55","labels_url":"https://api.github.com/repos/flowersteam/explauto/issues/55/labels{/name}","comments_url":"https://api.github.com/repos/flowersteam/explauto/issues/55/comments","events_url":"https://api.github.com/repos/flowersteam/explauto/issues/55/events","html_url":"https://github.com/flowersteam/explauto/issues/55","id":123790395,"number":55,"title":"Import third_party error","user":{"login":"clement-moulin-frier","id":6000622,"avatar_url":"https://avatars.githubusercontent.com/u/6000622?v=3","gravatar_id":"","url":"https://api.github.com/users/clement-moulin-frier","html_url":"https://github.com/clement-moulin-frier","followers_url":"https://api.github.com/users/clement-moulin-frier/followers","following_url":"https://api.github.com/users/clement-moulin-frier/following{/other_user}","gists_url":"https://api.github.com/users/clement-moulin-frier/gists{/gist_id}","starred_url":"https://api.github.com/users/clement-moulin-frier/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clement-moulin-frier/subscriptions","organizations_url":"https://api.github.com/users/clement-moulin-frier/orgs","repos_url":"https://api.github.com/users/clement-moulin-frier/repos","events_url":"https://api.github.com/users/clement-moulin-frier/events{/privacy}","received_events_url":"https://api.github.com/users/clement-moulin-frier/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-24T10:51:53Z","updated_at":"2015-12-24T13:59:36Z","closed_at":null,"body":"Hi,\r\n\r\nRight after a fresh install of explauto, running:\r\n\r\n```\r\npython -c \"import explauto\"\r\n```\r\n\r\nraises the following error:\r\n\r\n```\r\nTraceback (most recent call last):\r\n  File \"<string>\", line 1, in <module>\r\n  File \"explauto/__init__.py\", line 7, in <module>\r\n    from .sensorimotor_model.sensorimotor_model import SensorimotorModel\r\n  File \"explauto/sensorimotor_model/__init__.py\", line 7, in <module>\r\n    module = importlib.import_module('explauto.sensorimotor_model.{}'.format(mod_name))\r\n  File \"/usr/lib/python2.7/importlib/__init__.py\", line 37, in import_module\r\n    __import__(name)\r\n  File \"explauto/sensorimotor_model/non_parametric.py\", line 5, in <module>\r\n    from ..third_party.models.models.learner import Learner\r\nImportError: No module named third_party.models.models.learner\r\n```\r\n\r\nFrom what I remember, we removed the `third_party` package more than a year ago. However, it seems that it is still used in `non_parametric.py`(in particular the `Learner` class).\r\n\r\n\r\n@pierre-rouanet: do you remember the status of this?\r\n@sebastien-forestier: do you still use the `non_parametric` module? (e.g. for your LWR+CMAES inverse model?). Or do you rely on another module for LWR?\r\n\r\n@jypuigbo: as a temporary fix you can:\r\n- install the library in developer mode, as [explained here](http://flowersteam.github.io/explauto/installation.html#as-a-developer). Then comment the problematic line\r\n```\r\n File \"explauto/sensorimotor_model/non_parametric.py\", line 5, in <module>\r\n    from ..third_party.models.models.learner import Learner\r\n```\r\nand reinstall again. The error should disappear (you won't be able to use the `non_parametric` module though. You can also check with @santmarti who recently installed it without mentioning this problem.\r\n\r\nCheers"},"comment":{"url":"https://api.github.com/repos/flowersteam/explauto/issues/comments/167115726","html_url":"https://github.com/flowersteam/explauto/issues/55#issuecomment-167115726","issue_url":"https://api.github.com/repos/flowersteam/explauto/issues/55","id":167115726,"user":{"login":"sebastien-forestier","id":11048486,"avatar_url":"https://avatars.githubusercontent.com/u/11048486?v=3","gravatar_id":"","url":"https://api.github.com/users/sebastien-forestier","html_url":"https://github.com/sebastien-forestier","followers_url":"https://api.github.com/users/sebastien-forestier/followers","following_url":"https://api.github.com/users/sebastien-forestier/following{/other_user}","gists_url":"https://api.github.com/users/sebastien-forestier/gists{/gist_id}","starred_url":"https://api.github.com/users/sebastien-forestier/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sebastien-forestier/subscriptions","organizations_url":"https://api.github.com/users/sebastien-forestier/orgs","repos_url":"https://api.github.com/users/sebastien-forestier/repos","events_url":"https://api.github.com/users/sebastien-forestier/events{/privacy}","received_events_url":"https://api.github.com/users/sebastien-forestier/received_events","type":"User","site_admin":false},"created_at":"2015-12-24T13:59:36Z","updated_at":"2015-12-24T13:59:36Z","body":"I worked on that issue in branch models.\r\nThere I included directly and simplified only the useful code from Learner, so we can still use LWLR, WNN, BFGS, CMAES.\r\nI suggest that we discuss merging the changes here: https://github.com/flowersteam/explauto/pull/56\r\nBy the time we merge them, you can use branch models to have sensorimotor models working, or do the trick suggested by @clement-moulin-frier.\r\nCheers"}}