From Friday, April 19th (11:00 PM CDT) through Saturday, April 20th (2:00 PM CDT), 2024, ni.com will undergo system upgrades that may result in temporary service interruption.
We appreciate your patience as we improve our online experience.
From Friday, April 19th (11:00 PM CDT) through Saturday, April 20th (2:00 PM CDT), 2024, ni.com will undergo system upgrades that may result in temporary service interruption.
We appreciate your patience as we improve our online experience.
10-28-2019 07:16 PM
Is NI aware that the inputs to the gamma distribution (scale b and shape c parameters) are reversed? This labview bug has cost me many hours of code development time!!!! It would be nice to see this rectified in future versions in the full development system. Also a rebate would be nice, but that won't happen.
10-29-2019 12:09 AM
Hello
Reporting bugs has always been difficult with NI but they have done some corrections based on the developer's bug reports. Click the below link and try to contact "Troubleshoot and Develop" team.
http://sine.ni.com/apps/we/nicc.call_me?p_action=country&p_lang_id=US
The rest is in their hands.
-Rahul
Hit KUDOS for Thanks
10-29-2019 11:35 AM
@miklovicd wrote:
Is NI aware that the inputs to the gamma distribution (scale b and shape c parameters) are reversed?
Plain LabVIEW does not have anything named "gamma probability distribution". Is this from a toolkit? Are you talking about CDF? Is this classic or NXG? Can you point us to the online help of the exact function you are talking about. Thanks!
10-29-2019 01:24 PM
Its in LabVIEW full development system > Math> Prob and Stats> Probability>continuous pdf (gamma option)
There is the same problem with CDF and inverse CDF
10-29-2019 01:42 PM
@altenbach:
Yes, he's talking about the Gamma Distribution found as a polymorphic choice of the Continuous CDF VI found buried on the Mathematics Palette. And, in fact, he is also correct that "b" is the "shape" parameter and "c" is the "scale". Other than that, the VI seems to be correct (it calls the Incomplete Gamma Function and with b and c reversed, looks like it "does the right thing"). Another "hint" is that "c" is the divisor of "x", as though it was "scaling the input variable" ...
Bob Schor
10-29-2019 02:04 PM
Yes everything works just fine if you reverse the b and c inputs. The point is that one doesn't know to do this until lots lots of debugging identifies labview as the culprit. Once you look at the labview code it is obvious what is what.
10-29-2019 02:42 PM
I've taken the liberty of calling NI to report this as a "Documentation Bug". As you can imagine, this has to be "checked" and a remediation plan developed. Personally, I think the most unambiguous way to fix this is to call the parameters "x", "shape", and "scale" (rather than "x", "b", and "c", because who knows what "b" and "c" mean, but "shape" and "scale" have a certain intuitive flavor about them).
Bob Schor
10-29-2019 03:02 PM
I am not very familiar with these functions and their underlying math, but it seems to me that this polymorphic set of VIs has other problems and confusing assignments. Lets have a look at the diagram "Beta CDF" from the same set:
There is an entire instance of a subVI just floating around in the lower left, not connected to anything. Hopefully the compiler will drop that during dead code elimination. Also the wiring and terminal names is quite confusing (x wired to a, a wired to x, and b wired to y). Maybe it's all correct, but a common naming scheme for terminals would probably eliminate a lot of confusion and avoid errors. 😄
10-29-2019 03:19 PM
I added an entry in the bug thread.
10-29-2019 03:58 PM
They are unambiguously called shape and scale in the documentation. It is just that they are not assigned correctly to the inputs. Hopefully bug fixes will only be in the documentation, otherwise software that is currently working because this bug has been accounted for will stop working.
I agree that shape and scale would be better input names than a and b.