closed review Filter Error Codes (Error Package)

[jgcode 322]

Maybe make the icon just as large as the normal clear icon...

But that goes against the reason for having a small compact VI i the first place

I am leaning towards this one.

Anyone got any comments on the VIs Jim posted? Can it be enhanced?

I am thinking that this satisfies the requirements of this review and that it should be used as is.

[GregSands 77]

But that goes against the reason for having a small compact VI i the first place

I am leaning towards this one.

Is the gap between the small icon and the wires intentional? It does visually indicate that errors are cleared, but at the same time looks "wrong".

[jgcode 322]

Is the gap between the small icon and the wires intentional? It does visually indicate that errors are cleared, but at the same time looks "wrong".

No, it is not.

It is talked about in the above posts tho, and we tried to minimize it.

[Wouter 11]

But that goes against the reason for having a small compact VI i the first place

I never said that I was in favour for the small compact VI

But anyway I think I would rather like to see the icon stretched out a little then having the little gabs.

[Francois Normandin 177]

How about this?

[asbo 156]

Crafty! I approve.

[Wouter 11]

[jgcode 322]

How about this?

Crafty! I approve.

Really? I posted that CP a few posts up and didn't get anywhere near the response for François! ...lol:

Yer, that was my though too from the initial CP I posted:

Or we could leave it out (no Upstream Error as per the example you posted)?

I am happy with that CP too. I have all the information I require. I will move this review to pending and close it in a few days.

Cheers

-JG

[Francois Normandin 177]

I think it's not about the connector pane, but rather the small "ears" I added to the icon

[jgcode 322]

I think it's not about the connector pane, but rather the small "ears" I added to the icon

Ah-hah! (Sorry, completely missed that when I looked at it!)

In that case I say "brilliant" too.

[Yair 224]

I'm with Wouter on this - just make the icon a bit wider. If you use François's trick, you will probably have problems with matching some of the wires, because they have a specific pattern and you would need to move the VI a couple of pixels this or that way to match it.

Here's a very quick example:

I think that this isn't too big and could probably be made better with a few minutes of polishing.

[Mellroth 65]

Anyone got any comments on the VIs Jim posted? Can it be enhanced?

I would move the Merge Error inside the Error case.

If "error in" doesn't contain any error the output should be the "upstream error", i.e. no need to merge.

/J

[jgcode 322]

Thanks everyone for their input - all non-functional comments and suggestions have been reviewed with respect to updating the code Jim posted.

This OpenG Review is now closed. Please start a new thread to discuss new changes to this VI. Please PM me if there are any issues with this thread.