{"action":"created","comment":{"url":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/comments/165315604","pull_request_review_id":93231721,"id":165315604,"diff_hunk":"@@ -0,0 +1,132 @@\n+# Sardana Follow-up Meeting - 2018/01/25\n+\n+Held on 2018/01/25 at 14:00\n+\n+## Agenda\n+\n+1. Presentation\n+  1.  Sardana objects references and the compatibility with Taurus4 (#668 - kick off)\n+  2.  Centralized logging for Sardana with Elastic\n+2. User Experience Report - Round Table\n+3. Collaborative Developments - Review\n+4. New Developments, Enhancements or Bug Fixes\n+5. Pending PR distribution and Jan17 release\n+6. Any Other Business\n+\n+## Minutes\n+\n+### 1. Presentation\n+#### 1.1  Sardana objects references and the compatibility with Taurus4\n+\n+- PR: [#668](https://github.com/sardana-org/sardana/pull/668)\n+- Because of the cache mechanism if you change the alias, you must make an init in the pool.\n+- Keep reference able to distinguish devices and not relying on the tango (FQTN).\n+- What if the tango host changes? Problem with memorized attributes. PR: [#667](https://github.com/sardana-org/sardana/issues/667)\n+- Keep ID's and references internally\n+- Internally, tango always uses FQTN. It would be good to keep the names unified.\n+- Convert the names from short names to full names? ... This could be provided by Taurus.\n+- In a measurement group? Elements should be named by alias.\n+- In the MacroServer, the container should use the full name internally.\n+- MacroServer and Pool offering the alias for users always, if possible\n+- Append or prepend device names to aliases to elements with the same names?\n+- Pool_name.alias?\n+\n+__Actions__\n+\n+- [ ] With several Pools shared in a MacroServer, check if the actual behavior is good enough or if it needs to be improved.\n+- [ ] Start with the most simple case and evolve from there.\n+\n+\n+#### 1.2  Centralized logging for Sardana with Elastic\n+\n+- ElastichSearch, LogStash and Kibanna used for this prototype.\n+- How to make it tango independent?\n+  - SardanaCustomSettings could be used but it's not so easy ...\n+- Initialization in the server\n+- At this moment, the solution is tango-dependent because it's using tango properties for keeping the configuration\n+\n+__Actions__\n+- [ ] Zibi to make a PR and it will be evaluated\n+\n+\n+### 2. User Experience Report - Round Table\n+\n+__MAXIV__\n+- Sardana updated in all the beamlines except one.\n+- Users complaining about deprecation warnings in spock.\n+  - SardanaCustomSettings will be used for avoiding the Deprecation warnings.\n+\n+__SOLARIS__\n+- Massive upgrade for the control system\n+- MotorGroups\n+  - How to delete the MotorGroups? From Jive it may cause problems.\n+  Maybe some elements were removed at the same time? That could cause the errors.\n+  The reason for removing them was that maybe MotorGroups were causing troubles with overhead.\n+  It's a good idea to remove MotorGroups from time to time, they will be recreated if they are needed in future operations\n+- Events problems: Tango Events no received at all. Due to rebooting VMs. Event system stops working. Rebooting the VM again, fix it.\n+  Tango Forum: reported a list of possible problems related with Tango Events.\n+\n+__DESY__\n+- In shutdown, so no feedback from users.\n+- Planning the update of Sardana.\n+\n+__ALBA__\n+- Moving a pseudomotor hangs a macro or MacroServer (randomly). Seems like the State event is not generated. Pseudomotor based on two physical motor. It will be debugged next Monday.\n+  - Something similar saw at Solaris when the first movement of a pseudo was loosing the events (just after restart the pool). It will be reviewed by Gregorzs.\n+  - Position of physical motors is sometimes not updated and that causes the pseudo position not being well calculated.\n+- It would be useful to have a prepare-measurement (with number of repetitions) before the measurement, just once. (REPEATED from previous meeting)\n+- It would be useful to have a call to cleanup-measurement at the end of the measurement, not at each step in step scan, just once. (Repeated)\n+- Standard macros to manage the measurement groups\n+- New repeat parameters syntax in sequencer (from text file)\n+- New Macro API method that would be always called at the end of the macro execution, could be called \"cleanup\" or \"restore\"\n+  - Can it be solved with a general hook? generalOnStop hook?\n+\n+__Actions__\n+\n+- [ ] Share macros for managing the measurement groups and make a standard macros for this.\n+\n+\n+### 3. Collaborative Developments - Review\n+\n+__ALBA__\n+- About positions on Continuous scans, it couldn't be started.\n+\n+__MAXIV__\n+- We were more focused on updating Sardana and Taurus in the beamlines. Not much more done.\n+\n+__DESY__\n+- PR created.\n+\n+__SOLARIS__\n+- Working mostly on updates.\n+\n+__Common__\n+\n+- General Hooks pending from feedback from MAXIV and SOLARIS\n+\n+- Motor limits issue.\n+  - Investigation done for accessing the limits of a motor.\n+  - Avoid the use of attribute proxy and use the configuration of the attribute?\n+- Taurusform compatibility with names. SOLARIS interested in working in this issue, but they are very busy with the upgrades.","path":"20180125/MINUTES.md","position":110,"original_position":110,"commit_id":"91d5ca8fccb66c1bff309750cfc6545e900c0107","original_commit_id":"91d5ca8fccb66c1bff309750cfc6545e900c0107","user":{"login":"reszelaz","id":6735649,"avatar_url":"https://avatars1.githubusercontent.com/u/6735649?v=4","gravatar_id":"","url":"https://api.github.com/users/reszelaz","html_url":"https://github.com/reszelaz","followers_url":"https://api.github.com/users/reszelaz/followers","following_url":"https://api.github.com/users/reszelaz/following{/other_user}","gists_url":"https://api.github.com/users/reszelaz/gists{/gist_id}","starred_url":"https://api.github.com/users/reszelaz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/reszelaz/subscriptions","organizations_url":"https://api.github.com/users/reszelaz/orgs","repos_url":"https://api.github.com/users/reszelaz/repos","events_url":"https://api.github.com/users/reszelaz/events{/privacy}","received_events_url":"https://api.github.com/users/reszelaz/received_events","type":"User","site_admin":false},"body":"Taurus4 instead of Taurusform","created_at":"2018-02-01T10:40:07Z","updated_at":"2018-02-01T10:43:35Z","html_url":"https://github.com/sardana-org/sardana-followup/pull/7#discussion_r165315604","pull_request_url":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/7","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/comments/165315604"},"html":{"href":"https://github.com/sardana-org/sardana-followup/pull/7#discussion_r165315604"},"pull_request":{"href":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/7"}}},"pull_request":{"url":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/7","id":166248448,"html_url":"https://github.com/sardana-org/sardana-followup/pull/7","diff_url":"https://github.com/sardana-org/sardana-followup/pull/7.diff","patch_url":"https://github.com/sardana-org/sardana-followup/pull/7.patch","issue_url":"https://api.github.com/repos/sardana-org/sardana-followup/issues/7","number":7,"state":"open","locked":false,"title":"[WIP] Add meeting minutes","user":{"login":"amilan","id":4625755,"avatar_url":"https://avatars1.githubusercontent.com/u/4625755?v=4","gravatar_id":"","url":"https://api.github.com/users/amilan","html_url":"https://github.com/amilan","followers_url":"https://api.github.com/users/amilan/followers","following_url":"https://api.github.com/users/amilan/following{/other_user}","gists_url":"https://api.github.com/users/amilan/gists{/gist_id}","starred_url":"https://api.github.com/users/amilan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amilan/subscriptions","organizations_url":"https://api.github.com/users/amilan/orgs","repos_url":"https://api.github.com/users/amilan/repos","events_url":"https://api.github.com/users/amilan/events{/privacy}","received_events_url":"https://api.github.com/users/amilan/received_events","type":"User","site_admin":false},"body":"This commit adds the minutes for the meeting.","created_at":"2018-01-31T12:38:53Z","updated_at":"2018-02-01T10:43:35Z","closed_at":null,"merged_at":null,"merge_commit_sha":"533e3c662faf45f114830ad667612a72ac6f7c1a","assignee":null,"assignees":[],"requested_reviewers":[{"login":"daneos","id":2900361,"avatar_url":"https://avatars0.githubusercontent.com/u/2900361?v=4","gravatar_id":"","url":"https://api.github.com/users/daneos","html_url":"https://github.com/daneos","followers_url":"https://api.github.com/users/daneos/followers","following_url":"https://api.github.com/users/daneos/following{/other_user}","gists_url":"https://api.github.com/users/daneos/gists{/gist_id}","starred_url":"https://api.github.com/users/daneos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daneos/subscriptions","organizations_url":"https://api.github.com/users/daneos/orgs","repos_url":"https://api.github.com/users/daneos/repos","events_url":"https://api.github.com/users/daneos/events{/privacy}","received_events_url":"https://api.github.com/users/daneos/received_events","type":"User","site_admin":false}],"requested_teams":[],"milestone":null,"commits_url":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/7/commits","review_comments_url":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/7/comments","review_comment_url":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/comments{/number}","comments_url":"https://api.github.com/repos/sardana-org/sardana-followup/issues/7/comments","statuses_url":"https://api.github.com/repos/sardana-org/sardana-followup/statuses/91d5ca8fccb66c1bff309750cfc6545e900c0107","head":{"label":"amilan:2018_01_25_meeting","ref":"2018_01_25_meeting","sha":"91d5ca8fccb66c1bff309750cfc6545e900c0107","user":{"login":"amilan","id":4625755,"avatar_url":"https://avatars1.githubusercontent.com/u/4625755?v=4","gravatar_id":"","url":"https://api.github.com/users/amilan","html_url":"https://github.com/amilan","followers_url":"https://api.github.com/users/amilan/followers","following_url":"https://api.github.com/users/amilan/following{/other_user}","gists_url":"https://api.github.com/users/amilan/gists{/gist_id}","starred_url":"https://api.github.com/users/amilan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amilan/subscriptions","organizations_url":"https://api.github.com/users/amilan/orgs","repos_url":"https://api.github.com/users/amilan/repos","events_url":"https://api.github.com/users/amilan/events{/privacy}","received_events_url":"https://api.github.com/users/amilan/received_events","type":"User","site_admin":false},"repo":{"id":118473586,"name":"sardana-followup","full_name":"amilan/sardana-followup","owner":{"login":"amilan","id":4625755,"avatar_url":"https://avatars1.githubusercontent.com/u/4625755?v=4","gravatar_id":"","url":"https://api.github.com/users/amilan","html_url":"https://github.com/amilan","followers_url":"https://api.github.com/users/amilan/followers","following_url":"https://api.github.com/users/amilan/following{/other_user}","gists_url":"https://api.github.com/users/amilan/gists{/gist_id}","starred_url":"https://api.github.com/users/amilan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amilan/subscriptions","organizations_url":"https://api.github.com/users/amilan/orgs","repos_url":"https://api.github.com/users/amilan/repos","events_url":"https://api.github.com/users/amilan/events{/privacy}","received_events_url":"https://api.github.com/users/amilan/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/amilan/sardana-followup","description":"Minutes and other stuff gathered during the Sardana project follow-up meetings","fork":true,"url":"https://api.github.com/repos/amilan/sardana-followup","forks_url":"https://api.github.com/repos/amilan/sardana-followup/forks","keys_url":"https://api.github.com/repos/amilan/sardana-followup/keys{/key_id}","collaborators_url":"https://api.github.com/repos/amilan/sardana-followup/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/amilan/sardana-followup/teams","hooks_url":"https://api.github.com/repos/amilan/sardana-followup/hooks","issue_events_url":"https://api.github.com/repos/amilan/sardana-followup/issues/events{/number}","events_url":"https://api.github.com/repos/amilan/sardana-followup/events","assignees_url":"https://api.github.com/repos/amilan/sardana-followup/assignees{/user}","branches_url":"https://api.github.com/repos/amilan/sardana-followup/branches{/branch}","tags_url":"https://api.github.com/repos/amilan/sardana-followup/tags","blobs_url":"https://api.github.com/repos/amilan/sardana-followup/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/amilan/sardana-followup/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/amilan/sardana-followup/git/refs{/sha}","trees_url":"https://api.github.com/repos/amilan/sardana-followup/git/trees{/sha}","statuses_url":"https://api.github.com/repos/amilan/sardana-followup/statuses/{sha}","languages_url":"https://api.github.com/repos/amilan/sardana-followup/languages","stargazers_url":"https://api.github.com/repos/amilan/sardana-followup/stargazers","contributors_url":"https://api.github.com/repos/amilan/sardana-followup/contributors","subscribers_url":"https://api.github.com/repos/amilan/sardana-followup/subscribers","subscription_url":"https://api.github.com/repos/amilan/sardana-followup/subscription","commits_url":"https://api.github.com/repos/amilan/sardana-followup/commits{/sha}","git_commits_url":"https://api.github.com/repos/amilan/sardana-followup/git/commits{/sha}","comments_url":"https://api.github.com/repos/amilan/sardana-followup/comments{/number}","issue_comment_url":"https://api.github.com/repos/amilan/sardana-followup/issues/comments{/number}","contents_url":"https://api.github.com/repos/amilan/sardana-followup/contents/{+path}","compare_url":"https://api.github.com/repos/amilan/sardana-followup/compare/{base}...{head}","merges_url":"https://api.github.com/repos/amilan/sardana-followup/merges","archive_url":"https://api.github.com/repos/amilan/sardana-followup/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/amilan/sardana-followup/downloads","issues_url":"https://api.github.com/repos/amilan/sardana-followup/issues{/number}","pulls_url":"https://api.github.com/repos/amilan/sardana-followup/pulls{/number}","milestones_url":"https://api.github.com/repos/amilan/sardana-followup/milestones{/number}","notifications_url":"https://api.github.com/repos/amilan/sardana-followup/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/amilan/sardana-followup/labels{/name}","releases_url":"https://api.github.com/repos/amilan/sardana-followup/releases{/id}","deployments_url":"https://api.github.com/repos/amilan/sardana-followup/deployments","created_at":"2018-01-22T15:11:35Z","updated_at":"2017-10-26T15:55:57Z","pushed_at":"2018-01-31T12:44:59Z","git_url":"git://github.com/amilan/sardana-followup.git","ssh_url":"git@github.com:amilan/sardana-followup.git","clone_url":"https://github.com/amilan/sardana-followup.git","svn_url":"https://github.com/amilan/sardana-followup","homepage":null,"size":63,"stargazers_count":0,"watchers_count":0,"language":null,"has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":null,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"sardana-org:master","ref":"master","sha":"636972c297c7d95a51f9b045d28a5c41f75886d7","user":{"login":"sardana-org","id":22996613,"avatar_url":"https://avatars3.githubusercontent.com/u/22996613?v=4","gravatar_id":"","url":"https://api.github.com/users/sardana-org","html_url":"https://github.com/sardana-org","followers_url":"https://api.github.com/users/sardana-org/followers","following_url":"https://api.github.com/users/sardana-org/following{/other_user}","gists_url":"https://api.github.com/users/sardana-org/gists{/gist_id}","starred_url":"https://api.github.com/users/sardana-org/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sardana-org/subscriptions","organizations_url":"https://api.github.com/users/sardana-org/orgs","repos_url":"https://api.github.com/users/sardana-org/repos","events_url":"https://api.github.com/users/sardana-org/events{/privacy}","received_events_url":"https://api.github.com/users/sardana-org/received_events","type":"Organization","site_admin":false},"repo":{"id":108433408,"name":"sardana-followup","full_name":"sardana-org/sardana-followup","owner":{"login":"sardana-org","id":22996613,"avatar_url":"https://avatars3.githubusercontent.com/u/22996613?v=4","gravatar_id":"","url":"https://api.github.com/users/sardana-org","html_url":"https://github.com/sardana-org","followers_url":"https://api.github.com/users/sardana-org/followers","following_url":"https://api.github.com/users/sardana-org/following{/other_user}","gists_url":"https://api.github.com/users/sardana-org/gists{/gist_id}","starred_url":"https://api.github.com/users/sardana-org/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sardana-org/subscriptions","organizations_url":"https://api.github.com/users/sardana-org/orgs","repos_url":"https://api.github.com/users/sardana-org/repos","events_url":"https://api.github.com/users/sardana-org/events{/privacy}","received_events_url":"https://api.github.com/users/sardana-org/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/sardana-org/sardana-followup","description":"Minutes and other stuff gathered during the Sardana project follow-up meetings","fork":false,"url":"https://api.github.com/repos/sardana-org/sardana-followup","forks_url":"https://api.github.com/repos/sardana-org/sardana-followup/forks","keys_url":"https://api.github.com/repos/sardana-org/sardana-followup/keys{/key_id}","collaborators_url":"https://api.github.com/repos/sardana-org/sardana-followup/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/sardana-org/sardana-followup/teams","hooks_url":"https://api.github.com/repos/sardana-org/sardana-followup/hooks","issue_events_url":"https://api.github.com/repos/sardana-org/sardana-followup/issues/events{/number}","events_url":"https://api.github.com/repos/sardana-org/sardana-followup/events","assignees_url":"https://api.github.com/repos/sardana-org/sardana-followup/assignees{/user}","branches_url":"https://api.github.com/repos/sardana-org/sardana-followup/branches{/branch}","tags_url":"https://api.github.com/repos/sardana-org/sardana-followup/tags","blobs_url":"https://api.github.com/repos/sardana-org/sardana-followup/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/sardana-org/sardana-followup/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/sardana-org/sardana-followup/git/refs{/sha}","trees_url":"https://api.github.com/repos/sardana-org/sardana-followup/git/trees{/sha}","statuses_url":"https://api.github.com/repos/sardana-org/sardana-followup/statuses/{sha}","languages_url":"https://api.github.com/repos/sardana-org/sardana-followup/languages","stargazers_url":"https://api.github.com/repos/sardana-org/sardana-followup/stargazers","contributors_url":"https://api.github.com/repos/sardana-org/sardana-followup/contributors","subscribers_url":"https://api.github.com/repos/sardana-org/sardana-followup/subscribers","subscription_url":"https://api.github.com/repos/sardana-org/sardana-followup/subscription","commits_url":"https://api.github.com/repos/sardana-org/sardana-followup/commits{/sha}","git_commits_url":"https://api.github.com/repos/sardana-org/sardana-followup/git/commits{/sha}","comments_url":"https://api.github.com/repos/sardana-org/sardana-followup/comments{/number}","issue_comment_url":"https://api.github.com/repos/sardana-org/sardana-followup/issues/comments{/number}","contents_url":"https://api.github.com/repos/sardana-org/sardana-followup/contents/{+path}","compare_url":"https://api.github.com/repos/sardana-org/sardana-followup/compare/{base}...{head}","merges_url":"https://api.github.com/repos/sardana-org/sardana-followup/merges","archive_url":"https://api.github.com/repos/sardana-org/sardana-followup/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/sardana-org/sardana-followup/downloads","issues_url":"https://api.github.com/repos/sardana-org/sardana-followup/issues{/number}","pulls_url":"https://api.github.com/repos/sardana-org/sardana-followup/pulls{/number}","milestones_url":"https://api.github.com/repos/sardana-org/sardana-followup/milestones{/number}","notifications_url":"https://api.github.com/repos/sardana-org/sardana-followup/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/sardana-org/sardana-followup/labels{/name}","releases_url":"https://api.github.com/repos/sardana-org/sardana-followup/releases{/id}","deployments_url":"https://api.github.com/repos/sardana-org/sardana-followup/deployments","created_at":"2017-10-26T15:55:57Z","updated_at":"2017-10-26T15:55:57Z","pushed_at":"2018-01-31T12:45:00Z","git_url":"git://github.com/sardana-org/sardana-followup.git","ssh_url":"git@github.com:sardana-org/sardana-followup.git","clone_url":"https://github.com/sardana-org/sardana-followup.git","svn_url":"https://github.com/sardana-org/sardana-followup","homepage":null,"size":521,"stargazers_count":0,"watchers_count":0,"language":null,"has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"archived":false,"open_issues_count":2,"license":null,"forks":3,"open_issues":2,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/7"},"html":{"href":"https://github.com/sardana-org/sardana-followup/pull/7"},"issue":{"href":"https://api.github.com/repos/sardana-org/sardana-followup/issues/7"},"comments":{"href":"https://api.github.com/repos/sardana-org/sardana-followup/issues/7/comments"},"review_comments":{"href":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/7/comments"},"review_comment":{"href":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/sardana-org/sardana-followup/pulls/7/commits"},"statuses":{"href":"https://api.github.com/repos/sardana-org/sardana-followup/statuses/91d5ca8fccb66c1bff309750cfc6545e900c0107"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"reszelaz"}}