{"action":"created","issue":{"url":"https://api.github.com/repos/linuxmao-org/Frontieres/issues/54","repository_url":"https://api.github.com/repos/linuxmao-org/Frontieres","labels_url":"https://api.github.com/repos/linuxmao-org/Frontieres/issues/54/labels{/name}","comments_url":"https://api.github.com/repos/linuxmao-org/Frontieres/issues/54/comments","events_url":"https://api.github.com/repos/linuxmao-org/Frontieres/issues/54/events","html_url":"https://github.com/linuxmao-org/Frontieres/pull/54","id":381862490,"node_id":"MDExOlB1bGxSZXF1ZXN0MjMxNzI3NTQy","number":54,"title":"Olof : ajout fenetre de parametres des clouds (touche \"p\")","user":{"login":"olof29","id":25823124,"node_id":"MDQ6VXNlcjI1ODIzMTI0","avatar_url":"https://avatars3.githubusercontent.com/u/25823124?v=4","gravatar_id":"","url":"https://api.github.com/users/olof29","html_url":"https://github.com/olof29","followers_url":"https://api.github.com/users/olof29/followers","following_url":"https://api.github.com/users/olof29/following{/other_user}","gists_url":"https://api.github.com/users/olof29/gists{/gist_id}","starred_url":"https://api.github.com/users/olof29/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/olof29/subscriptions","organizations_url":"https://api.github.com/users/olof29/orgs","repos_url":"https://api.github.com/users/olof29/repos","events_url":"https://api.github.com/users/olof29/events{/privacy}","received_events_url":"https://api.github.com/users/olof29/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":13,"created_at":"2018-11-17T13:17:52Z","updated_at":"2018-11-19T07:09:09Z","closed_at":"2018-11-18T19:30:43Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/linuxmao-org/Frontieres/pulls/54","html_url":"https://github.com/linuxmao-org/Frontieres/pull/54","diff_url":"https://github.com/linuxmao-org/Frontieres/pull/54.diff","patch_url":"https://github.com/linuxmao-org/Frontieres/pull/54.patch"},"body":"la mise à jour se fait dans les deux sens (modifier dans la fenetre modifie aussi les parametres dans la scene, et modifier les parametres de la scene modifie ceux de la fenêtre)\r\nune fenetre possible par cloud.\r\nlà ou je ne suis pas sur du coup c'est sur la question de la destruction de la fenetre. devrais je declarer le pointeur vers la fenetre en unique ptr, afin qu'elle soit detruite à la destruction du cloud, ou cela est il deja le cas ? ou faut il detruire celle ci a la fermeture, et devoir la recrer à chaque fois qu'on veut la voir ?"},"comment":{"url":"https://api.github.com/repos/linuxmao-org/Frontieres/issues/comments/439792274","html_url":"https://github.com/linuxmao-org/Frontieres/pull/54#issuecomment-439792274","issue_url":"https://api.github.com/repos/linuxmao-org/Frontieres/issues/54","id":439792274,"node_id":"MDEyOklzc3VlQ29tbWVudDQzOTc5MjI3NA==","user":{"login":"olof29","id":25823124,"node_id":"MDQ6VXNlcjI1ODIzMTI0","avatar_url":"https://avatars3.githubusercontent.com/u/25823124?v=4","gravatar_id":"","url":"https://api.github.com/users/olof29","html_url":"https://github.com/olof29","followers_url":"https://api.github.com/users/olof29/followers","following_url":"https://api.github.com/users/olof29/following{/other_user}","gists_url":"https://api.github.com/users/olof29/gists{/gist_id}","starred_url":"https://api.github.com/users/olof29/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/olof29/subscriptions","organizations_url":"https://api.github.com/users/olof29/orgs","repos_url":"https://api.github.com/users/olof29/repos","events_url":"https://api.github.com/users/olof29/events{/privacy}","received_events_url":"https://api.github.com/users/olof29/received_events","type":"User","site_admin":false},"created_at":"2018-11-19T07:09:09Z","updated_at":"2018-11-19T07:09:09Z","author_association":"CONTRIBUTOR","body":"tout cela est généré par la mise à jour dans un timer. je ne comprends vraiment pas cette méthode : \r\n- visiblement ça induitde sérieux problemes dans la saisie des valeurs\r\n- ça déclenche des mises à jour à intervalles réguliers, qu'il y ait eu modifications ou pas ? \r\nsi oui, ça mange forcement du temps de calcul inutilement.\r\n- pour moi, gerer la mise à jour de la fenetre dans un timer n'est pas bon.\r\nsi tu veux absolument demenager le declenchement de la mise à jour, il faut qu'elle soit déclenchée aux appels des fonctions de cloud qui font les set..parametre du cloud.\r\n- si on ne le fait pas dans la fonction, il faut donc le faire juste apres chacun des appels à celle ci...\r\n- je ne vois donc aucun intéret à l'avoir deplacé... à part le fait que maintenant le cloud n'a plus acces à la fenetre lui correspondant.\r\n\r\n- je ne saisis toujours pas pourquoi il faut absolument decoupler le visuel du model : \r\nen terme de logique de programmation, j'ai toujours appris à raisonner selon la méthode qui consite à se demander \"qui appartient à qui\", ou qui \"dépend de qui\", or clairement la fenetre des parametres d'un cloud dépend du cloud, puisqu'on a permis d'en avoir plusieurs, une par cloud.\r\nsi au depart, à ma question : \"en faut il une seule ou plusieurs ?\", on avait répondu \"une seule\", là j'aurais compris cette logique de dire que la fenetre ne dépend pas du cloud, seul le contenu dépend d'elle, mais ici c'est clairement le cas.\r\n\r\n- tu dis que cela peut poser des problemes si modele et visuels sont liés. je ne comprends pas quel probleme cela poserait. est ce parcequ'une mise à jour en continu venant de l'exterieur comme un changement de volume en continu envoyé par le controle OSC, par exemple, declencherait trop de mises à jour visuelles, et risquerait de generer des xruns ? dans ce cas, ne pourrait on pas l'eviter en faisant simplement des mises à jour plus ciblées ? (là la mise a jour concerne toute la fenetre, on peut tres bien envisager de ne mettre à jour que les controles concernés, par exemple)\r\n\r\n"}}
{"actor":{"display_login":"olof29"}}