[Soc-2017-dev] Weekly Report #10 - Normal Editing Tools

[Rohan Rathi]

Thank you very much :D

On Thu, Aug 24, 2017 at 9:56 PM, Howard Trickey <howard.trickey at gmail.com>
wrote:

> Nice report and nice work on your GSoC project, Rohan.
>
> On Thu, Aug 24, 2017 at 12:06 PM Rohan Rathi <rohanrathi08 at gmail.com>
> wrote:
>
>> Hey,
>> I've created the final report. Could you have a look at it before I hand
>> it over for final submission?
>> Thank you :)
>>
>> Final Report: https://wiki.blender.org/index.php/User:RohanRathi/
>> GSoC_2017/Final_Report
>>
>>
>> On Mon, Aug 14, 2017 at 9:26 PM, Rohan Rathi <rohanrathi08 at gmail.com>
>> wrote:
>>
>>> Hey,
>>> I've removed the use of the flag (haven't removed the flag yet) and have
>>> singled out most of the bugs in the code. Just some small UI changes
>>> required and minor changes to the modifier and tools have been suggested,
>>> not too hard to handle.
>>> I'm beginning work on the user and code documentation on the wiki and
>>> prepare for final evaluation.
>>>
>>> On Mon, Aug 7, 2017 at 4:21 PM, Bastien Montagne <montagne29 at wanadoo.fr>
>>> wrote:
>>>
>>>> Hi Rohan,
>>>>
>>>> Agree polishing and bugfixing should be top priority now.
>>>>
>>>> Did quick review of code (nothing in-depth yet), overall looks very
>>>> good. Noted the following points that you are still using
>>>> BMO_OPTYPE_FLAG_INVALIDATE_CLNOR_ALL in addition to
>>>> BMO_OPTYPE_FLAG_NORMALS_CALC, already said it, but imho this is needlessly
>>>> verbose - I can’t imagine any case where you'd want to invalidate basic
>>>> normals, while keeping clnors one valid? So imho,
>>>> BMO_OPTYPE_FLAG_NORMALS_CALC should be redefined to also contain
>>>> BMO_OPTYPE_FLAG_INVALIDATE_CLNOR_ALL, such that using the former
>>>> automatically also triggers the later, without having to change any code
>>>> for that.
>>>>
>>>> Cheers,
>>>> Bastien
>>>> Le 04/08/2017 à 20:12, Rohan Rathi a écrit :
>>>>
>>>> *Would really appreciate feedback on the tools added and the weighted
>>>> normal modifier. Latest Windows release is included.*
>>>>
>>>> What I did this week:
>>>> I've added some functionality to the weighted normals modifier. It had
>>>> some serious bugs and glitches so  I've spent the majority of my time to
>>>> refine and work on the features I've added. Most likely, the modifier will
>>>> work completely fine now.
>>>>
>>>> Added the following:
>>>> 1) Modifier now respects sharp edges similar to sharp edges with smooth
>>>> shading.
>>>> 2) Added vertex group support. Modifier will only affect vertex groups
>>>> and leave all other loop data unchanged.
>>>> 3) Add smooth/sharp flags as boolean weights. A check as well whether
>>>> to use this function.
>>>>
>>>> What I plan on doing next week:
>>>> With just over 2 weeks left for the project. I feel adding more tools
>>>> is not a good option, not that I have anything important left to add. I'll
>>>> go on to maintaining and documenting the code from next week onwards and
>>>> prepare for final eval. There are some bugs and tasks that need attention
>>>> so I'll put on focus on those.
>>>>
>>>> Latest Windows Release: https://drive.google.com/file/d/
>>>> 0B9gCAchc4CLIV0lISFU5ZWptNXM/view?usp=sharing
>>>>
>>>> Links:
>>>> My proposal: https://wiki.blender.org/index.php/User:
>>>> RohanRathi/GSoC_2017/Proposal
>>>> Documentation: https://docs.google.com/document/d/
>>>> 1YXOLuiYjUgaoxawtXqcRkET-G2M3MbASVop88lKaKbg/edit
>>>> Git branch: https://developer.blender.org/diffusion/B/
>>>> browse/soc-2017-normal-tools/
>>>> Blender Artists Thread: https://blenderartists.org/forum/
>>>> showthread.php?427746-GSoC-2017-Normal-Editing-Tools
>>>>
>>>> Thank You,
>>>> Rohan Rathi
>>>>
>>>>
>>>> _______________________________________________
>>>> Soc-2017-dev mailing listSoc-2017-dev at blender.orghttps://lists.blender.org/mailman/listinfo/soc-2017-dev
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Soc-2017-dev mailing list
>>>> Soc-2017-dev at blender.org
>>>> https://lists.blender.org/mailman/listinfo/soc-2017-dev
>>>>
>>>>
>>>
>> _______________________________________________
>> Soc-2017-dev mailing list
>> Soc-2017-dev at blender.org
>> https://lists.blender.org/mailman/listinfo/soc-2017-dev
>>
>
> _______________________________________________
> Soc-2017-dev mailing list
> Soc-2017-dev at blender.org
> https://lists.blender.org/mailman/listinfo/soc-2017-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.blender.org/pipermail/soc-2017-dev/attachments/20170825/e61c9c20/attachment.html>