Jump to content

Remove the "(error in)" text from default error controls


Recommended Posts

QUOTE (Aristos Queue @ Feb 25 2009, 11:15 AM)

Shouldn't error terminals be moved -- on all nodes -- to be the top terminal of all nodes? , wouldn't that be a better strategy?

I like this idea!

I would like to see the default label "(no error)" part of the error in go away.

Link to comment

QUOTE (gleichman @ Feb 25 2009, 01:56 PM)

I would like to see the default label "(no error)" part of the error in go away.

It takes about a minute to go into C:\Program Files\National Instruments\LabVIEW 8.6\vi.lib\errclust.llb and remove the annoying "(no error)" part of the label (don't forget to fix both the classic and the 3D version). That's just about the second thing I do to a fresh copy of LabVIEW after turning off 'Lock Automatic Tool Selection', 'Automatic Wire Routing', and AutoGrow.

Link to comment

QUOTE (jdunham @ Feb 25 2009, 07:07 PM)

It takes about a minute to go into C:\Program Files\National Instruments\LabVIEW 8.6\vi.lib\errclust.llb and remove the annoying "(no error)" part of the label (don't forget to fix both the classic and the 3D version). That's just about the second thing I do to a fresh copy of LabVIEW after turning off 'Lock Automatic Tool Selection', 'Automatic Wire Routing', and AutoGrow.

:worship:

Same routine here but I did NOT knw about the llb.

Ben

Link to comment

QUOTE (jdunham @ Feb 25 2009, 04:07 PM)

It takes about a minute to go into C:\Program Files\National Instruments\LabVIEW 8.6\vi.lib\errclust.llb and remove the annoying "(no error)" part of the label (don't forget to fix both the classic and the 3D version)...

I actually do not mind the "(no error)", but I do mind the size of the error cluster. I change them back to the size they use to be in older LV version (significantly smaller) suing the same procedure Jason described.

PJM

Link to comment

QUOTE (PJM_labview @ Feb 26 2009, 09:01 AM)

I actually do not mind the "(no error)", but I do mind the size of the error cluster. I change them back to the size they use to be in older LV version (significantly smaller) using the same procedure Jason described.

It's not such a big deal on the front panel, but I believe its removal makes the Context Help window (Ctrl-H) look a lot better. I'm fine with default values in parentheses when they are useful, but when they don't add any information, then they shouldn't be there. Maybe the original idea was that first time users might think that wiring anything into that input would signify an error, but it's more likely that "it seemed like a good idea at the time".

Link to comment

QUOTE (jdunham @ Feb 26 2009, 01:10 PM)

I'm fine with default values in parentheses when they are useful, but when they don't add any information, then they shouldn't be there.

That's a really good point. I only use the default value in parentheses if the default value of the control isn't the default value of the datatype.

Link to comment

QUOTE (crelf @ Feb 26 2009, 10:26 AM)

That's a really good point. I only use the default value in parentheses if the default value of the control isn't the default value of the datatype.

Well it's also fine if the default needs clarification or has a special meaning like

filter coefficient(0:no filtering)

overwrite?(F:Append to file)

While I'm on my soapbox, making inputs "required" is a really good thing, and for the most part, required inputs don't need to have their defaults mentioned at all. But that has the same exception, where you may need to clarify a special interpretation of the default value for that data type.

Link to comment

QUOTE (jdunham @ Feb 26 2009, 01:47 PM)

I was thinking "I'll post a reply agreeing with that", but by the time my browser came up I think I'd changed my mind :)

I agree in principle that clarifying meaning is a good thing to do, but I'm not sure I agree with the examples you gave. I, of course, don't know the examples intimately, but here goes:

Sorry about getting on my soapbox, but I've never found where this sort of thing couldn't have been replaced with more intuative design.

QUOTE (jdunham @ Feb 26 2009, 01:47 PM)

While I'm on my soapbox, making inputs "required" is a really good thing, and for the most part, required inputs don't need to have their defaults mentioned at all.

I sure hope there's room on your soapbox for me too :) The only time I'd want a default mentioned when a connector pane element is required is when it's being run as a top level VI and it calls the "reint to default" method somewhere. We need to be mindful that it's not the control that we're making required, but the connection to it when it's used as a subVI.

Link to comment

QUOTE (crelf @ Feb 26 2009, 11:50 AM)

I was thinking "I'll post a reply agreeing with that", but by the time my browser came up I think I'd changed my mind :)

Well I agree with you! :thumbup: My examples were just off the top of my head, since I have (*gasp*) other work to do.

The 'overwrite/append' example is straight from the old file I/O VIs and I have to say, the the new version that always overwrites is a big step backward (getting off topic...)

I would agree that if an input requires a lot of explanatory labeling, there's probably a bad design somewhere around there.

What this thread is really about is that words that add no extra value should not be on your VIs, and I get peeved by style guides that apply those kind of rules like "Always put the default value in parens in the label" or "always make inputs required rather than recommended" or always use "error in/error out" [even when it's totally unnecessary].

Link to comment

QUOTE (jdunham @ Feb 26 2009, 01:07 AM)

It takes about a minute to go into C:\Program Files\National Instruments\LabVIEW 8.6\vi.lib\errclust.llb and remove the annoying "(no error)" part of the label (don't forget to fix both the classic and the 3D version). That's just about the second thing I do to a fresh copy of LabVIEW after turning off 'Lock Automatic Tool Selection', 'Automatic Wire Routing', and AutoGrow.

The fun thing is, you can create an OGP file (either with VIPM or OpenG Package builder) to overwrite that specific llb, just take the OGP's with you run VIPM and tada.

Ton

Link to comment

QUOTE (crelf @ Feb 26 2009, 09:50 PM)

filter coefficient(0:no filtering) <- if this is an input to a VI that filters, then you shouldn't be using this VI at all if no filtering is required. Make a decision outside of it and case it out if not required. Otherwise, all you're doing is moving that case decision to inside the subVI, which is hiding functionality, or, in this case, non-functionality :)

Well, I happen to disagree with that one. The whole point of a subVI is encapsulating functionality, so why put a case structure around every single instance where I wouldn't want to execute the code? This takes time, takes space, creates noise and actually hides the functionality more IMO, since you have to look inside the other frame to make sure nothing happens there. You're not hiding the functionality if you say "NO filtering" in your input.

The real problem here is using a numeric instead of an enum. I'm guessing that this comes from an old NI VI.

Link to comment

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.