AllenInstitute / MIES

Multichannel Igor Electrophysiology Suite
https://alleninstitute.github.io/MIES/user.html
Other
23 stars 7 forks source link

AI_MIESAutoPipetteOffset looks odd #1326

Open t-b opened 2 years ago

t-b commented 2 years ago

While working on using constants for decimal multiplier conversion (#1316) I found https://github.com/AllenInstitute/MIES/blob/3c1d97f1ae6da919552251cc23ab544a2e0b39ca/Packages/MIES/MIES_AmplifierInteraction.ipf#L710-L730.

Some questions:

The code goes back to 54cf23d4 (Added a MIES function to zero the DC current in Vclamp to use in place of the slow MCC operation auto pipette offset, 2015-08-29).

t-b commented 2 years ago

The current code is correct for VC

•printf "%f\r" 140* 500 / 1000
  70.000000
•printf "%f\r" 140e-12* 500e6 *1000
  70.000000