[Bf-cycles] Node names (two can have seemly same name and appearance)
Lukas Tönne
lukas.toenne at gmail.com
Sun Nov 3 14:55:58 CET 2013
Discussed with Philip on IRC:
* I don't really see the need for type strings in the labels, but if it's
really wanted i think it should be added secondary to the current label,
i.e. "Add (Math)" or so instead of "Math - Add". The left side with the
more detailed and useful info is then always visible even if strings get
cut off.
* Changing node size automatically as in the patch should be avoided imo -
width is defined by the user in the first place. It could be an operator
"Fit optimal node size" or so. At least it should be an optional setting,
maybe next to the snapping buttons in the header (snapping will also get
some improvements to make it work on the grid)
* The way labels are returned in the code atm does not allow dynamic
strings (returns const char*). The suggested patch would create a memory
leak. I want to extend the node API for labels anway to allow using dynamic
labels in python nodes, i'll make sure to make this more flexible then.
On Sun, Nov 3, 2013 at 2:05 PM, Philipp Oeser <info at graphics-engineer.com>wrote:
> Sorry, previous pic was a bit misleading (changed the operation _only_
> on the collapsed nodes). This one is more consistent:
>
> http://www.pasteall.org/pic/show.php?id=61839
>
>
> On Sun, 2013-11-03 at 13:46 +0100, Philipp Oeser wrote:
> > Not sure about the two liner, but what we can do is draw the (collapsed)
> > node a bit wider when the nodelabel gets longer? (patch below has
> > functionality that does this when you collapse a node)
> >
> > http://www.pasteall.org/pic/61837
> > (patch here http://www.pasteall.org/46982)
> >
> > Greetz
> > Philipp
> >
> > P.S.: this doesnt seem to be specifically Cycles related btw. Its more
> > general NodeEditor stuff. Maybe lukas_t can comment?
> >
> >
> > On Sun, 2013-11-03 at 09:55 +0100, Constantin Rahn wrote:
> > > I like this solution.
> > >
> > > Would it be possible to make it a two liner, with a break in case of
> > > collapsed nodes?
> > >
> > > Happy blending
> > > Conz
> > >
> > > Am 03.11.2013 00:35, schrieb Philipp Oeser:
> > > > Hm, gave it a quick go, not sure if this is a step forward.
> > > > The default collapsed node seems too small to show _both_ nodetype
> and
> > > > operation (even though when the collapsed node is resized in width it
> > > > works quite well...) Anyways, if in doubt, I would prefer operation
> as
> > > > well.
> > > >
> > > > Have a look here: http://www.pasteall.org/pic/61802
> > > > (patch here if someone wants to experiment more with this
> > > > http://www.pasteall.org/46976)
> > > >
> > > > Greetz
> > > > Philipp
> > > >
> > > >
> > > > On Sat, 2013-11-02 at 21:00 +0100, Gottfried Hofmann wrote:
> > > >> On 11/02/2013 07:42 PM, Greg Zaal wrote:
> > > >>> Could we just make the automatic label something like "Mix -
> Multiply"?
> > > >> That is a very good idea indeed!
> > > >>
> > > >> Gottfried
> > > >>
> > > >>
> > > >>> I'm actually not particularly bothered by this, people who get
> confused
> > > >>> by it are only confused once, and then know it. In this case I'd
> say
> > > >>> favour the experienced users and leave it as-is, or compromise and
> make
> > > >>> everyone happy with the cost of a longer label.
> > > >>>
> > > >>>
> > > >>> On 2 November 2013 19:18, Piotr Adamowicz <padamowicz at gmail.com
> > > >>> <mailto:padamowicz at gmail.com>> wrote:
> > > >>>
> > > >>> If the problem is making it easy for beginners to find the
> nodes, then
> > > >>> perhaps the solution would be to move the math node into its
> own
> > > >>> section and add a menu item for each of its modes while
> keeping the
> > > >>> math node itself as is? That way the beginners would find it
> easily
> > > >>> and the regular users would still keep their useful labels.
> > > >>>
> > > >>> On Sat, Nov 2, 2013 at 6:13 PM, ishly habib <
> ishrox123 at gmail.com
> > > >>> <mailto:ishrox123 at gmail.com>> wrote:
> > > >>> > I think that it would be enough if the search would
> understand
> > > >>> add as a
> > > >>> > version of the mix node and pop-up because the typical
> thing for
> > > >>> a beginner
> > > >>> > to do when I can't find the node is to search it. The
> solution of
> > > >>> headers is
> > > >>> > clunky in my opinion. if that matters
> > > >>> >
> > > >>> > On 01/11/2013 10:38 pm, "Brecht Van Lommel"
> > > >>> <brechtvanlommel at pandora.be <mailto:
> brechtvanlommel at pandora.be>>
> > > >>> > wrote:
> > > >>> >>
> > > >>> >> I'm not sure I agree, if you build a setup with math or
> color mix
> > > >>> >> nodes it's quite tedious to have to rename them manually
> I would
> > > >>> >> think? Although perhaps this is a non-issue for typical
> users,
> > > >>> just in
> > > >>> >> my experience I quite like this renaming.
> > > >>> >>
> > > >>> >> Perhaps we should have tooltips when you are over a node
> (header),
> > > >>> >> with the node name and description of the node, that
> would solve the
> > > >>> >> issue in a different way.
> > > >>> >>
> > > >>> >> Brecht.
> > > >>> >>
> > > >>> >> On Fri, Nov 1, 2013 at 6:07 AM, Greg Zaal <
> gregzzmail at gmail.com
> > > >>> <mailto:gregzzmail at gmail.com>> wrote:
> > > >>> >> > Hey David,
> > > >>> >> >
> > > >>> >> > I agree, it's especially difficult for beginners to
> look at
> > > >>> someone's
> > > >>> >> > node
> > > >>> >> > setup and wonder where this weird "Modulo" node (for
> example)
> > > >>> comes from
> > > >>> >> > that they can't find in the menus. The title of the
> node should be
> > > >>> >> > displayed
> > > >>> >> > exactly how it is in the Add menu, and only change when
> the user
> > > >>> >> > explicitly
> > > >>> >> > changes it's label in the properties region.
> > > >>> >> >
> > > >>> >> > It's not cycles specifically, but also the compositing
> (and I
> > > >>> assume
> > > >>> >> > texture) nodes as well that have this issue.
> > > >>> >> >
> > > >>> >> > Cheers,
> > > >>> >> > Greg
> > > >>> >> >
> > > >>> >> > On Nov 1, 2013 6:08 AM, "David Black" <
> db4tech at yahoo.co.uk
> > > >>> <mailto:db4tech at yahoo.co.uk>> wrote:
> > > >>> >> >>
> > > >>> >> >> Apologies, I do have dyslexia, subject should read...
> > > >>> seemingly not
> > > >>> >> >> seemly.
> > > >>> >> >>
> > > >>> >> >> The following line should read...
> > > >>> >> >> (While nodes are collapsed) it becomes possible for
> math and
> > > >>> vector
> > > >>> >> >> math
> > > >>> >> >> nodes to have a seemingly identical name and
> appearance.
> > > >>> >> >>
> > > >>> >> >> I notice the two nodes in question have different
> socket
> > > >>> inputs which
> > > >>> >> >> helps differentiate to some extent, still like how
> other
> > > >>> nodes provide
> > > >>> >> >> header names rather than function, this in itself is
> of course
> > > >>> >> >> subjective.
> > > >>> >> >>
> > > >>> >> >> David
> > > >>> >> >> --
> > > >>> >> >>
> > > >>> >> >> 3d-designs-davidblack.blogspot.com
> > > >>> <http://3d-designs-davidblack.blogspot.com>
> > > >>> >> >>
> > > >>> >> >> On 01/11/2013 03:38, David Black wrote:
> > > >>> >> >>
> > > >>> >> >> Hi Brecht and other mailing list members.
> > > >>> >> >>
> > > >>> >> >> While we are discussing nodes, what are peoples
> thoughts on
> > > >>> >> >> Converters\Math node displaying 'Math' and
> Converters\Vector Math
> > > >>> >> >> displaying
> > > >>> >> >> 'Vector Math' for their node names?
> > > >>> >> >>
> > > >>> >> >> They are the only two nodes which change name
> depending on their
> > > >>> >> >> function,
> > > >>> >> >> while I can see this is useful, it can also cause
> identification
> > > >>> >> >> difficulties. (While nodes are collapsed) it becomes
> possible
> > > >>> for math
> > > >>> >> >> and
> > > >>> >> >> vector math nodes to have an identical name and
> appearance.
> > > >>> >> >>
> > > >>> >> >> With all other nodes displaying header names rather
> than
> > > >>> function,
> > > >>> >> >> would
> > > >>> >> >> adding header names to these two nodes help with
> > > >>> identification and
> > > >>> >> >> maintain
> > > >>> >> >> consistency?
> > > >>> >> >>
> > > >>> >> >> Thank you for your time,
> > > >>> >> >> David
> > > >>> >> >> --
> > > >>> >> >>
> > > >>> >> >> 3d-designs-davidblack.blogspot.com
> > > >>> <http://3d-designs-davidblack.blogspot.com>
> > > >>> >> >>
> > > >>> >> >>
> > > >>> >> >>
> > > >>> >> >> _______________________________________________
> > > >>> >> >> Bf-cycles mailing list
> > > >>> >> >> Bf-cycles at blender.org <mailto:Bf-cycles at blender.org>
> > > >>> >> >> http://lists.blender.org/mailman/listinfo/bf-cycles
> > > >>> >> >>
> > > >>> >> >>
> > > >>> >> >>
> > > >>> >> >> _______________________________________________
> > > >>> >> >> Bf-cycles mailing list
> > > >>> >> >> Bf-cycles at blender.org <mailto:Bf-cycles at blender.org>
> > > >>> >> >> http://lists.blender.org/mailman/listinfo/bf-cycles
> > > >>> >> >>
> > > >>> >> >
> > > >>> >> > _______________________________________________
> > > >>> >> > Bf-cycles mailing list
> > > >>> >> > Bf-cycles at blender.org <mailto:Bf-cycles at blender.org>
> > > >>> >> > http://lists.blender.org/mailman/listinfo/bf-cycles
> > > >>> >> >
> > > >>> >> _______________________________________________
> > > >>> >> Bf-cycles mailing list
> > > >>> >> Bf-cycles at blender.org <mailto:Bf-cycles at blender.org>
> > > >>> >> http://lists.blender.org/mailman/listinfo/bf-cycles
> > > >>> >
> > > >>> >
> > > >>> > _______________________________________________
> > > >>> > Bf-cycles mailing list
> > > >>> > Bf-cycles at blender.org <mailto:Bf-cycles at blender.org>
> > > >>> > http://lists.blender.org/mailman/listinfo/bf-cycles
> > > >>> >
> > > >>>
> > > >>>
> > > >>>
> > > >>> --
> > > >>> Piotr "MadMinstrel" Adamowicz
> > > >>> padamowicz at gmail.com <mailto:padamowicz at gmail.com>
> > > >>> _______________________________________________
> > > >>> Bf-cycles mailing list
> > > >>> Bf-cycles at blender.org <mailto:Bf-cycles at blender.org>
> > > >>> http://lists.blender.org/mailman/listinfo/bf-cycles
> > > >>>
> > > >>>
> > > >>>
> > > >>>
> > > >>> _______________________________________________
> > > >>> Bf-cycles mailing list
> > > >>> Bf-cycles at blender.org
> > > >>> http://lists.blender.org/mailman/listinfo/bf-cycles
> > > >>>
> > > >>
> > > >
> > > > _______________________________________________
> > > > Bf-cycles mailing list
> > > > Bf-cycles at blender.org
> > > > http://lists.blender.org/mailman/listinfo/bf-cycles
> > > >
> > >
> > > _______________________________________________
> > > Bf-cycles mailing list
> > > Bf-cycles at blender.org
> > > http://lists.blender.org/mailman/listinfo/bf-cycles
> >
> >
> > _______________________________________________
> > Bf-cycles mailing list
> > Bf-cycles at blender.org
> > http://lists.blender.org/mailman/listinfo/bf-cycles
>
>
> _______________________________________________
> Bf-cycles mailing list
> Bf-cycles at blender.org
> http://lists.blender.org/mailman/listinfo/bf-cycles
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.blender.org/pipermail/bf-cycles/attachments/20131103/69f686b4/attachment.htm
More information about the Bf-cycles
mailing list