<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=utf-8"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman",serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
{mso-style-priority:34;
margin-top:0cm;
margin-right:0cm;
margin-bottom:0cm;
margin-left:36.0pt;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman",serif;}
span.EstiloCorreo18
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;
font-family:"Calibri",sans-serif;
mso-fareast-language:EN-US;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]--></head><body lang=ES-AR link=blue vlink=purple><div class=WordSection1><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>Hey there, <o:p></o:p></span></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'><o:p> </o:p></span></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>Some comments Gpencil related<o:p></o:p></span></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'><o:p> </o:p></span></p><p class=MsoNormal><b><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>- "continuous drawing is enabled using double D key"<o:p></o:p></span></b></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>Actually what is most bother me with Gpencil as artist is that we are forced to hold a key while drawing (D). <o:p></o:p></span></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>My proposal is that the user only needs to press D button to start in Gpencil and only in "continuos" mode; for me it makes little sense to have a tool that stop working after the first stroke you made. (we talk today with Daniel about it and he is going to test this approach and come back with comments)<o:p></o:p></span></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'><o:p> </o:p></span></p><p class=MsoNormal><b><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>- Use ctrl/alt for straight line drawing<o:p></o:p></span></b></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>It Works really nice! really handy<o:p></o:p></span></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'><o:p> </o:p></span></p><p class=MsoNormal><b><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>-Ctrl-H for Hiding Points<o:p></o:p></span></b></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>It works well and is useful mainly having the alpha option to tweak visibility<o:p></o:p></span></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'><o:p> </o:p></span></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>Keep the good work coming!<o:p></o:p></span></p><p class=MsoNormal><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'><o:p> </o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>Cheers<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>Matias<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'><o:p> </o:p></span></p><p class=MsoNormal><b><span lang=ES style='font-size:11.0pt;font-family:"Calibri",sans-serif'>De:</span></b><span lang=ES style='font-size:11.0pt;font-family:"Calibri",sans-serif'> bf-animsys-bounces@blender.org [mailto:bf-animsys-bounces@blender.org] <b>En nombre de </b>Daniel Martinez Lara<br><b>Enviado el:</b> lunes, 18 de julio de 2016 9:36 a. m.<br><b>Para:</b> Discussion list to assist animation developers <bf-animsys@blender.org><br><b>Asunto:</b> Re: [Bf-animsys] Fwd: GP v2 - Code Review<o:p></o:p></span></p><p class=MsoNormal><o:p> </o:p></p><div><div><div><div><p class=MsoNormal style='margin-bottom:12.0pt'>Hi Joshua! (and and all)<o:p></o:p></p></div><p class=MsoNormal style='margin-bottom:12.0pt'>About your questions about the<b> Draw </b>(Dkey) and<b> continous drawing</b> (D+D) solution, I'm happy with this solution, because you dont need set/decide before, just choose you mode on de fly.<o:p></o:p></p></div><p class=MsoNormal style='margin-bottom:12.0pt'>And about <b>Ctrl-H for Hiding Points</b> in happry too, because when your are making fine tune tweaking/sculpting you need see your lines as is, and with the vertex dots you can't.<o:p></o:p></p></div><p class=MsoNormal>Daniel<o:p></o:p></p><div><p class=MsoNormal style='margin-bottom:12.0pt'><br><br><br><o:p></o:p></p></div></div><div><p class=MsoNormal><o:p> </o:p></p><div><p class=MsoNormal>On Sun, Jul 17, 2016 at 5:21 PM, Joshua Leung <<a href="mailto:aligorith@gmail.com" target="_blank">aligorith@gmail.com</a>> wrote:<o:p></o:p></p><blockquote style='border:none;border-left:solid #CCCCCC 1.0pt;padding:0cm 0cm 0cm 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0cm;margin-bottom:5.0pt'><div><div><p class=MsoNormal style='margin-bottom:12.0pt'>Let's move future discussions about GP dev to the bf-animsys list :)<o:p></o:p></p></div><p class=MsoNormal>As I proposed during today's meeting, I'd like to propose that we split the changes into 3-sets: <br> 1) simple stuff that can go in regardless of the other changes (e.g. the stuff under #2 - there are a few other things introduced recently that also fit in here too), <br> 2) the layers vs colors stuff - this is the main bulk of the patch, and will need the most careful checking, just to make sure that we're all happy with it (since it will be quite a large change), <br> 3) stuff which touches other parts of Blender and needs further discussion with the other module teams (e.g. compositor integration)<o:p></o:p></p><div><p class=MsoNormal style='margin-bottom:12.0pt'><o:p> </o:p></p></div><div><p class=MsoNormal>Cheers,<o:p></o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'>Joshua<o:p></o:p></p></div><div><div><div><div><div><p class=MsoNormal><o:p> </o:p></p><div><div><p class=MsoNormal style='margin-bottom:12.0pt'>---------- Forwarded message ----------<br>From: <b>Joshua Leung</b> <<a href="mailto:aligorith@gmail.com" target="_blank">aligorith@gmail.com</a>><br>Date: Mon, Jul 18, 2016 at 2:28 AM<br>Subject: GP v2 - Code Review<br>To: antonioya blend <<a href="mailto:blendergit@gmail.com" target="_blank">blendergit@gmail.com</a>><br>Cc: Matias Mendiola <<a href="mailto:info@mendiobox.com" target="_blank">info@mendiobox.com</a>>, Daniel Martínez Lara <<a href="mailto:web@pepeland.com" target="_blank">web@pepeland.com</a>><br><br><o:p></o:p></p><div><div><div><div><div><div><div><p class=MsoNormal style='margin-bottom:12.0pt'>Hi Antonio,<o:p></o:p></p></div><p class=MsoNormal style='margin-bottom:12.0pt'>I thought it would be good to go through the changes that have been happening in the GP v2 branch so far, to make sure that everything is going on track for inclusion in master later down the track.<o:p></o:p></p></div><p class=MsoNormal style='margin-bottom:12.0pt'>So, here are some comments from what I've seen so far...<o:p></o:p></p></div><p class=MsoNormal>Cheers,<o:p></o:p></p></div><p class=MsoNormal style='margin-bottom:12.0pt'>Joshua<br><br>=============================================<br><br><b>1)</b> <b>"continuous drawing is enabled using double D key"</b><br><a href="https://git.blender.org/gitweb/gitweb.cgi/blender-staging.git/commit/d619edf64c24a19110ee190e911de360fb9b0dea" target="_blank">https://git.blender.org/gitweb/gitweb.cgi/blender-staging.git/commit/d619edf64c24a19110ee190e911de360fb9b0dea</a><o:p></o:p></p></div><p class=MsoNormal style='margin-bottom:12.0pt'>Interesting idea, and I hope that it works out in testing. However, when I've tried these kinds of key mappings in the past, I've found that it's a very bad idea to try mixing "hold-drag" hotkeys (e.g. D-LMBdrag) vs double-press (e.g. D-D). <br><br>The issue mainly comes about if you usually keep the DKEY held down, release a little bit after doing a normal stroke, then, <i>very quickly</i> press DKEY again and try to draw a second stroke ==> What's likely to happen is that before you manage to get the pen back down for the second stroke, the double-press handling has already fired, AND it is likely that it might've also started drawing straight away (so the line will have low pressure, be in a bad position, and also be slightly wonky for a bit). What ends up happening is that it start feeling quite flaky.<o:p></o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'>Anyway, just mentioning this so that you guys are aware of it and can test it out further.<o:p></o:p></p></div><div><p class=MsoNormal>[EDIT/CORRECTION: I just checked the code for this. My only concern is that this is probably going to just push the event handling for Grease Pencil drawing past the point where it gets unmanageable. To be fair, it's actually quite bad already :) But I think that it may be a good idea if we first did some cleanup here in master (i.e. separately from all the exciting new stuff going on here in this branch), to lay the foundations for what's coming up next).]<o:p></o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'><br><br><o:p></o:p></p></div><div><p class=MsoNormal><b>2) There are quite a few tweaks which I'd be happy to include in master now (if Daniel and Matias are satisfied that they work well).</b> From the looks of things, these are general usability tweaks that don't really depend on the big features anyway, so we can start with these first since it's easier.<br><br>Clear candidates for inclusion:<br> - "<b>Use ctrl/alt for straight line drawing</b>" - Sounds like a good idea. I'll test it a bit and to check how well it works<o:p></o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'> - "<b>Invert F/Ctrl-F</b>" - Let me know how well this works. IIRC, this is one of the things that I'm not really that happy with currently, so if it all works well, that'll be great.<br> <o:p></o:p></p></div><div><p class=MsoNormal>Candidates which might need a little closer review first:<o:p></o:p></p></div><div><p class=MsoNormal> - <b>Joining strokes together</b> - This is something that would be great to have. It's great to see that you've implemented it already, so I won't have to, haha :D<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal> - <b>"Ctrl-H for Hiding Points"</b> - My big question would have to be - how well is this working for you guys (Daniel/Matias?). <o:p></o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'> (On a side note, it's better to just do "<span style='font-family:"Courier New"'>ToolSettings *ts = CTX_data_tool_settings(C)</span>;" instead of accessing via Scene, if you don't/won't need access to anything in scene. Also, if you can get scene (which is almost always guaranteed), then toolsettings will never be null. And IIRC, it might be better to use one of the poll callbacks for editable strokes instead of the "editmode_toggle" one)<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal><b>3) Grease Pencil and Compositor</b><o:p></o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'>While it's nice to see this functionality, I strongly suggest that you run double-check with the other key devs who deal with this stuff (in particular Sergey, but probably also Campbell, Severin, and Ton) that the approach taken aligns with future development plans for both the layers system, but also the viewport project and/or Blender Internal modernisation projects. For instance, I remember Ton saying that he was keen for there to be a more general "OpenGL pass" in the compositor instead of just a "Grease Pencil" specific solution.<o:p></o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'>IIRC, last time this came up, it just so happened that most of the key people here were also away on holiday that week (or weeks). <br><br><o:p></o:p></p></div><div><p class=MsoNormal><b>4) Brush Presets</b><o:p></o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'>Looking at this a bit, I suspect that it might be better to look into defining a set of these in Python scripts (like the other presets)?<br><br><o:p></o:p></p></div><div><p class=MsoNormal><b>5) Object/Bone Parenting</b><o:p></o:p></p></div><div><p class=MsoNormal style='margin-bottom:12.0pt'>First, it's great that we've finally got this in place :D<o:p></o:p></p></div><div><p class=MsoNormal>The main thing I think we need to work on here is more about how this integrates into Blender - specifically, how this fits with the normal rigging pipeline and/or integrates with other objects. I think I mentioned this last time already, but IMO "layer transformed by object" should by default happen if the GP datablock is attached to an object. (Maybe we can still keep the current object/bone properties for usage with scene-linked GP datablocks?) <br><br>That way, it fits into the normal object-handling system of Blender more. Once you do that, you could have it so that hiding the object shows/hides the GP stuff associated with it, and later on, the restrict render option could be used to control whether the GP strokes get included in the render (via autoconverting the GP strokes to curves during the "Blender data to renderdata conversion" - e.g. 'convertblender.c' or the equivalent for cycles)<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div></div></div><p class=MsoNormal><o:p> </o:p></p></div></div></div></div></div></div></div><p class=MsoNormal style='margin-bottom:12.0pt'><br>_______________________________________________<br>Bf-animsys mailing list<br><a href="mailto:Bf-animsys@blender.org">Bf-animsys@blender.org</a><br><a href="https://lists.blender.org/mailman/listinfo/bf-animsys" target="_blank">https://lists.blender.org/mailman/listinfo/bf-animsys</a><o:p></o:p></p></blockquote></div><p class=MsoNormal><br><br clear=all><br>-- <o:p></o:p></p><div><p class=MsoNormal>----------------------------------------<br> Daniel Martinez Lara<o:p></o:p></p><div><p class=MsoNormal><a href="https://twitter.com/_pepeland_" target="_blank">http://twitter.com/_pepeland_</a><o:p></o:p></p></div><div><p class=MsoNormal><br> <a href="http://www.pepeland.com" target="_blank">www.pepeland.com</a><br><a href="http://www.pepe-school-land.com" target="_blank">www.pepe-school-land.com</a><br>----------------------------------------<o:p></o:p></p></div></div></div></div></body></html>