{"action":"created","issue":{"url":"https://api.github.com/repos/LabPy/lantz_drivers/issues/1","labels_url":"https://api.github.com/repos/LabPy/lantz_drivers/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/LabPy/lantz_drivers/issues/1/comments","events_url":"https://api.github.com/repos/LabPy/lantz_drivers/issues/1/events","html_url":"https://github.com/LabPy/lantz_drivers/issues/1","id":59464193,"number":1,"title":"Package organisation","user":{"login":"MatthieuDartiailh","id":4481740,"avatar_url":"https://avatars.githubusercontent.com/u/4481740?v=3","gravatar_id":"","url":"https://api.github.com/users/MatthieuDartiailh","html_url":"https://github.com/MatthieuDartiailh","followers_url":"https://api.github.com/users/MatthieuDartiailh/followers","following_url":"https://api.github.com/users/MatthieuDartiailh/following{/other_user}","gists_url":"https://api.github.com/users/MatthieuDartiailh/gists{/gist_id}","starred_url":"https://api.github.com/users/MatthieuDartiailh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MatthieuDartiailh/subscriptions","organizations_url":"https://api.github.com/users/MatthieuDartiailh/orgs","repos_url":"https://api.github.com/users/MatthieuDartiailh/repos","events_url":"https://api.github.com/users/MatthieuDartiailh/events{/privacy}","received_events_url":"https://api.github.com/users/MatthieuDartiailh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-03-02T10:37:33Z","updated_at":"2015-07-24T02:55:12Z","closed_at":null,"body":"- Not a major question but how should we organize this package ?\r\n\r\n  I would be in favour of organizing by vendor independently of the underlying protocol (ie visa, a dll, another python library). \r\n- A secondary question is what to do with hp/agilent/keysight instruments and other similar stuff ? \r\n- Third point as the metaclass magic can increase the lodaing time of the drivers it might be interesting to know all the existing drivers without having to import everything, and import only when the user needs a specific driver. I would love to have such a feature as when building uis I need to propose all existing drivers to my user but I like my app to start fast."},"comment":{"url":"https://api.github.com/repos/LabPy/lantz_drivers/issues/comments/124298589","html_url":"https://github.com/LabPy/lantz_drivers/issues/1#issuecomment-124298589","issue_url":"https://api.github.com/repos/LabPy/lantz_drivers/issues/1","id":124298589,"user":{"login":"hgrecco","id":278566,"avatar_url":"https://avatars.githubusercontent.com/u/278566?v=3","gravatar_id":"","url":"https://api.github.com/users/hgrecco","html_url":"https://github.com/hgrecco","followers_url":"https://api.github.com/users/hgrecco/followers","following_url":"https://api.github.com/users/hgrecco/following{/other_user}","gists_url":"https://api.github.com/users/hgrecco/gists{/gist_id}","starred_url":"https://api.github.com/users/hgrecco/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hgrecco/subscriptions","organizations_url":"https://api.github.com/users/hgrecco/orgs","repos_url":"https://api.github.com/users/hgrecco/repos","events_url":"https://api.github.com/users/hgrecco/events{/privacy}","received_events_url":"https://api.github.com/users/hgrecco/received_events","type":"User","site_admin":false},"created_at":"2015-07-24T02:55:12Z","updated_at":"2015-07-24T02:55:12Z","body":"The convention that we had until previous version of Lantz is the following:\r\n1. One folder for each company\r\n2. The folder is a (sub)package, contains and __init__ file which imports and exposes ALL the driver classes in the package. (This can be done manually or automated)\r\n3. The classes are named according to the module using Python PEP8. When the model starts with a number, we prefix with the first letter of the company. I am fine to choose another (short) prefix.\r\n4. The number of modules in the package is really dependent on what makes sense for the company.\r\n"}}