<br><br><div class="gmail_quote">On Mon, Oct 29, 2012 at 10:00 PM, <span dir="ltr"><<a href="mailto:bf-python-request@blender.org" target="_blank">bf-python-request@blender.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Send Bf-python mailing list submissions to<br>
<a href="mailto:bf-python@blender.org">bf-python@blender.org</a><br>
<br>
To subscribe or unsubscribe via the World Wide Web, visit<br>
<a href="http://lists.blender.org/mailman/listinfo/bf-python" target="_blank">http://lists.blender.org/mailman/listinfo/bf-python</a><br>
or, via email, send a message with subject or body 'help' to<br>
<a href="mailto:bf-python-request@blender.org">bf-python-request@blender.org</a><br>
<br>
You can reach the person managing the list at<br>
<a href="mailto:bf-python-owner@blender.org">bf-python-owner@blender.org</a><br>
<br>
When replying, please edit your Subject line so it is more specific<br>
than "Re: Contents of Bf-python digest..."<br>
<br>
<br>
Today's Topics:<br>
<br>
1. Re: contrib addons & candidates for release (Campbell Barton)<br>
2. Contrib Addon Review: add_mesh_building_objects (Campbell Barton)<br>
<br>
<br>
----------------------------------------------------------------------<br>
<br>
Message: 1<br>
Date: Mon, 29 Oct 2012 18:48:53 +1100<br>
From: Campbell Barton <<a href="mailto:ideasman42@gmail.com">ideasman42@gmail.com</a>><br>
Subject: Re: [Bf-python] contrib addons & candidates for release<br>
To: Blender Foundation Python list <<a href="mailto:bf-python@blender.org">bf-python@blender.org</a>><br>
Message-ID:<br>
<CAEcf3NzbAkeWZuNdmsrF20X=<a href="mailto:LuXh3EnjYF0ANj_-Z4NCmgpt6A@mail.gmail.com">LuXh3EnjYF0ANj_-Z4NCmgpt6A@mail.gmail.com</a>><br>
Content-Type: text/plain; charset=UTF-8<br>
<br>
Hi there, Im going over these addons for review today and will post<br>
codereview linkes here.<br>
<br>
Some of these addons need general functionality review before code<br>
review even, so will do that first.<br>
<br>
On Thu, Oct 25, 2012 at 11:15 AM, Brendon Murphy<br>
<<a href="mailto:meta.androcto1@gmail.com">meta.androcto1@gmail.com</a>> wrote:<br>
> hi,<br>
> Before I get to the contrib addons list for release consideration, I would<br>
> like to mention that it would not be hard to trim the lesser addons from<br>
> contrib & enable contrib for release.<br>
> Most users know of contrib addons & have trouble to install them in official<br>
> builds. Users are & can be made aware or reminded that contrib addons are<br>
> 'experimental' as they already are with the "Testing" button to view contrib<br>
> addons in the addons menu/s.<br>
> Not all contrib addons are suitable for release & i have been working hard<br>
> towards cleaning up contrib & preparing many addons for release.<br>
> This can be seen in the "Tools" where I group similar addons with the goal<br>
> of easing the amount of small similar function addons into menu's. This<br>
> works well in reducing the volume of addons & eases the menu bloat & helps<br>
> users save time.<br>
><br>
> Addons proposed to move from contrib to release.<br>
><br>
> add_mesh_building_objects<br>
> <a href="https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/add_mesh_building_objects/" target="_blank">https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/add_mesh_building_objects/</a><br>
> Maintainers: brikbot, meta-androcto<br>
> Status: Stable & working for many releases before it was merged/added to<br>
> contrib.<br>
> Known issues: needs code tips translated, may not use latest function calls<br>
><br>
> btrace<br>
> <a href="https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/btrace/__init__.py" target="_blank">https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/btrace/__init__.py</a><br>
> Maintainers: Crazycourier, Liero<br>
> Status: Stable & working for many releases<br>
> Known issues: has been code reviewed & may not use the best methods/function<br>
> calls<br>
><br>
> mesh_select_tools<br>
> <a href="https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/mesh_select_tools/" target="_blank">https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/mesh_select_tools/</a><br>
> Maintainers: meta-androcto, codemanx<br>
> Status: Stable & working for many releases before it was merged/added to<br>
> contrib.<br>
> Known issues: does not use latest function calls.<br>
><br>
> object_mangle tools<br>
> <a href="https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/object_mangle_tools.py" target="_blank">https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/object_mangle_tools.py</a><br>
> Maintainer: Phil Cote<br>
> Status: Stable & working for many releases<br>
> Known issues: does not use latest function calls.<br>
><br>
> object_edit_linked<br>
> <a href="https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/object_edit_linked.py" target="_blank">https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/object_edit_linked.py</a><br>
> Maintainer: Fweeb<br>
> Status: Stable<br>
> Known Issues: None<br>
><br>
> system_project_folder<br>
> <a href="https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/system_project_folder.py" target="_blank">https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/system_project_folder.py</a><br>
> Maintainer: Dany Lebel<br>
> Status: Stable<br>
> Known issues: none<br>
><br>
> ui_layer_manager<br>
> <a href="https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/ui_layer_manager.py" target="_blank">https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/ui_layer_manager.py</a><br>
> Maintainer: Alfonso Annarumma:<br>
> Status: Stable<br>
> Known issues: none<br>
><br>
> Addons worth considering:<br>
> <a href="https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/io_mesh_xyz/" target="_blank">https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/io_mesh_xyz/</a><br>
> by blendphys<br>
><br>
> <a href="https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/uv_bake_texture_to_vcols.py" target="_blank">https://svn.blender.org/svnroot/bf-extensions/contrib/py/scripts/addons/uv_bake_texture_to_vcols.py</a><br>
> by Patrick Boelens, CoDEmanX<br>
><br>
> please note:<br>
> I have also been working on several toolkits, merging lesser scripts &<br>
> grouping similar tools, Release & Contrib can use more improvement, as<br>
> toolskits/merges get done more proposed addons will follow :)<br>
><br>
> I would ask that Code reviewers take into account that these addons work &<br>
> are stable for many months, whilst the code & methods may not be perfect, it<br>
> can be improved during review or improved as issues arise.<br>
> There are possibly minor easy to fix issues such as wiki pages to do that<br>
> I'm quite happy to fix if needed.<br>
><br>
> Also Blender Developers & Addon Developers are very welcome to help &<br>
> propose addons fro release or merge. Please if you see addons that fall<br>
> withing your area of Blender, such as animation or sequencer or uv modules,<br>
> feel free to contact the author & make suggestions on how they should<br>
> proceed towards release.<br>
><br>
> Thanks.<br>
> Brendon Murphy (meta-androcto)<br>
> Blender Extensions Addons team<br>
><br>
><br>
><br>
><br>
><br>
> _______________________________________________<br>
> Bf-python mailing list<br>
> <a href="mailto:Bf-python@blender.org">Bf-python@blender.org</a><br>
> <a href="http://lists.blender.org/mailman/listinfo/bf-python" target="_blank">http://lists.blender.org/mailman/listinfo/bf-python</a><br>
><br>
<br>
<br>
<br>
--<br>
- Campbell<br>
<br>
<br>
------------------------------<br>
<br>
Message: 2<br>
Date: Mon, 29 Oct 2012 19:55:52 +1100<br>
From: Campbell Barton <<a href="mailto:ideasman42@gmail.com">ideasman42@gmail.com</a>><br>
Subject: [Bf-python] Contrib Addon Review: add_mesh_building_objects<br>
To: Blender Foundation Python list <<a href="mailto:bf-python@blender.org">bf-python@blender.org</a>><br>
Message-ID:<br>
<CAEcf3NwBXOyTM=<a href="mailto:bWREBPA_hoHQ-LMrMT-sGeyd0KGEpjdtzkqw@mail.gmail.com">bWREBPA_hoHQ-LMrMT-sGeyd0KGEpjdtzkqw@mail.gmail.com</a>><br>
Content-Type: text/plain; charset=UTF-8<br>
<br>
Hi Brendon & Addon Authors,<br>
<br>
To me these addons seem more like a parametric-model-library, then a<br>
general addons.<br>
<br>
Since blender doesn't support parametric objects I can see why<br>
developers wrote these, but Im also not convinced this<br>
is a good reason to add all of them into trunk.<br>
<br>
As examples of what you can do with addons these are quite<br>
interesting, but even if you were using blender<br>
to model buildings, I think only 1 or 2 of these would end up being useful.<br>
<br>
A small negative for including these addons at all is that I dont<br>
think you could use them together to make a<br>
building with any kind of consistancy - you would end up with a<br>
castle-wall, mixed with a minimalist window, some beams, a balcony +<br>
some nicely<br>
detailed stairs.<br>
<br>
To authors, hope you guys don't take offense to my review, if some<br>
users are successfully including these tools in their workflow id like<br>
to know about it & see results.<br>
<br>
<br>
<br>
####################<br>
Functionality Review<br>
####################<br>
<br>
<br>
Beam builder<br>
------------<br>
<br>
The shapes are so primitive that you wouldnt save a lot of time by<br>
using this addon.<br>
<br>
If you happen to be needing to make shapes like this a _lot_, likely<br>
you already have a small library of shapes you'll<br>
re-use, adjusting the length, taper etc on existing models is also trivial.<br>
<br>
There is still some use for these kinds of addons but I think its so<br>
specific that such addons can stay 3rd party/contrib.<br>
<br>
(IMHO, don't promote to trunk)<br>
<br>
<br>
Balcony<br>
-------<br>
<br>
Again, this is adds a really specific model + materials, which I don't<br>
think are generally useful.<br>
If - for example - you had a library of balconies, most of these could<br>
be added and heights adjusted - in a few seconds.<br>
<br>
<br>
If you're an artist making a building, probably you have some ideas of<br>
your own how to add a balcony, which isn't really<br>
such a complicated shape,<br>
<br>
Even so, I think this has more merit then the beam builder.<br>
<br>
note: dimensions as integers here isn't good IMHO.<br>
<br>
(IMHO, don't promote to trunk)<br>
<br>
<br>
Solve (Window Frame)<br>
--------------------<br>
<br>
Seems a bit limited, adds a single kind of window frame, which is OK,<br>
as far as that goes - but if I were using<br>
a tool like this I'd want it to have support for a few different kinds<br>
of windows (Maybe 6+ main categories).<br>
<br>
The current window types are quite simple and take <1min (rough<br>
guess). to model.<br>
<br>
(IMHO, don't promote to trunk)<br>
<br>
... but If this were a more general window-frame generator, Id consider it.<br>
<br>
<br>
Wall Factory<br>
------------<br>
<br>
This addon is pretty neat, unlike most others in this category - it<br>
could save quite a bit of time compared to making<br>
such a model by hand.<br>
<br>
But its also quite limited, maybe it should be called `Castle Wall` factory.<br>
<br>
Im tempted to include this for novelty value, its fun!<br>
<br>
But I think you either want a lot more detaul, or a lot less.<br>
If you made an entire castle - this is way too much detail... and if<br>
you make a single/section of a castle for a close up render, the<br>
bricks are obviously cubes.<br>
So Im scratching my head to find a use-case.<br>
<br>
Also, even if this was the exact level of detail for some project -<br>
you would want more control then this addon gives you.<br>
<br>
- The ability to place windows. have the wall follow a path. - have<br>
the stairs lead somewhere useful...<br>
<br>
So its cool but the functionality included seems a bit haphazard - the<br>
developer included some interesting stuff,<br>
but probably didnt model even a small section of a castle using it<br>
(maybe Im wrong here).<br>
<br>
(IMHO, don't promote to trunk)<br>
<br>
<br>
Stair Builder<br>
-------------<br>
<br>
This addon creates fairly detailed models that would for sure save<br>
some time - compared to making manually.<br>
It also includes enough styles to be generally useful IMHO.<br>
<br>
So, on functionality level I'd accept this, but noticed some oddities.<br>
<br>
- Added objects are not selected (add object operators should select<br>
all newly added objects).<br>
- This adds many objects, not just a single object, this could be<br>
annoying for users,<br>
perhaps add option to give single parent? Also, since this adds many<br>
matching objects it could use linked duplicates<br>
(every step is the same for eg).<br>
- Spiral staircase does not support hand-rail options - best disable<br>
these or add support.<br>
<br>
Would promote this to trunk if these issues are resolved.<br>
<br>
<br>
- Campbell<br>
<br>
<br>
------------------------------<br>
<br>
_______________________________________________<br>
Bf-python mailing list<br>
<a href="mailto:Bf-python@blender.org">Bf-python@blender.org</a><br>
<a href="http://lists.blender.org/mailman/listinfo/bf-python" target="_blank">http://lists.blender.org/mailman/listinfo/bf-python</a><br>
<br>
<br>
End of Bf-python Digest, Vol 92, Issue 4<br>
****************************************<br>
</blockquote></div><br><div><div>ideasman_42, hi, reading review on building objects,</div><div><br></div><div>I'm sure brikbot can fix stairs & you raised appropriates issues there, such as single object & spiral stairs not complete.</div>
<div><br></div><div>It looks like you missed Window, it's just been updated in svn & is much more powerful. very useful for shop fronts & not so detailed buildings, it now features arc, tri & sloping windows, other upgrades & correct translation.</div>
<div><br></div><div>Wall factory, or Castle wall as you correctly call, is not well maintained but popular, given the novelty it's quite good & given some effort, some half decent castles can be assembled using multiple wall types. I'm happy to see this stay in contrib until a suitable maintainer is available.</div>
</div><div><br></div><div>Thanks for the time reviewing, </div><div>Brendon. :)</div><div><br></div>