search

puhep / pudb

Purdue CMS FPix Database
0 stars 0 forks source link

BL_LL_905 Bare Module Leakage Current Pass/Fail is Wrong #65

Closed NickHinton closed 9 years ago

NickHinton commented 9 years ago

This module breaks down at -110V, and all the measured currents and voltages are negative. So it should fail both the slope and 150V checks, but it is listed as passing on the DB.

khan62 commented 9 years ago

There isn't any data after 110V, so it's probably considering I(V=150) = 0, which would make it pass both. I'll need to make it consider nonexistent values a different value; I think PHP has a constant for infinity that could be used here.

khan62 commented 9 years ago

It is now set so that if the value is not found for either 100V or 150V, for criteria purposes it is effectively set to infinity. This may be a problem if there isn't a value for 100V, as this would cause I(V=150)/I(V=100)<2 to always pass.

merkelp commented 9 years ago

I would think during production we don't want a grade for a module which does not have a proper iv measurement. It would be safer to set the grade to d or such as a warning.

Petra

Sent from my iPhone

On Sep 2, 2015, at 4:28 PM, Kamal Khan notifications@github.com wrote:

It is now set so that if the value is not found for either 100V or 150V, for criteria purposes it is effectively set to infinity. This may be a problem if there isn't a value for 100V, as this would cause I(V=150)/I(V=100)<2 to always pass.

— Reply to this email directly or view it on GitHub.