From c19e82e0e242d3dcea625d218b6595660e64ffde Mon Sep 17 00:00:00 2001 From: "W. Trevor King" Date: Thu, 7 May 2009 15:35:37 -0400 Subject: [PATCH] Added bugtracking with Bugs Everywhere. The bug I added it for turned out to be due to the calling software, not PyComedi itself. I also removed an old debugging printout from simult_aio.py. --- .../3eaa128a-84ff-4aad-909d-6921d6bdaa64/body | 4 ++ .../values | 8 +++ .../9b70d421-62c0-47ce-b648-93afb6a988f6/body | 37 +++++++++++++ .../values | 11 ++++ .../a02dd6cc-132c-438a-96b2-f3677673bdfe/body | 55 +++++++++++++++++++ .../values | 8 +++ .../values | 17 ++++++ .be/settings | 2 + .be/version | 1 + pycomedi/simult_aio.py | 1 - 10 files changed, 143 insertions(+), 1 deletion(-) create mode 100644 .be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/3eaa128a-84ff-4aad-909d-6921d6bdaa64/body create mode 100644 .be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/3eaa128a-84ff-4aad-909d-6921d6bdaa64/values create mode 100644 .be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/9b70d421-62c0-47ce-b648-93afb6a988f6/body create mode 100644 .be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/9b70d421-62c0-47ce-b648-93afb6a988f6/values create mode 100644 .be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/a02dd6cc-132c-438a-96b2-f3677673bdfe/body create mode 100644 .be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/a02dd6cc-132c-438a-96b2-f3677673bdfe/values create mode 100644 .be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/values create mode 100644 .be/settings create mode 100644 .be/version diff --git a/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/3eaa128a-84ff-4aad-909d-6921d6bdaa64/body b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/3eaa128a-84ff-4aad-909d-6921d6bdaa64/body new file mode 100644 index 0000000..16833df --- /dev/null +++ b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/3eaa128a-84ff-4aad-909d-6921d6bdaa64/body @@ -0,0 +1,4 @@ +Fixed in unfold.py + +When adjusting the stepper (too close or too far), relative setpoint +was not being taken into account. Now it is. diff --git a/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/3eaa128a-84ff-4aad-909d-6921d6bdaa64/values b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/3eaa128a-84ff-4aad-909d-6921d6bdaa64/values new file mode 100644 index 0000000..87ce99a --- /dev/null +++ b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/3eaa128a-84ff-4aad-909d-6921d6bdaa64/values @@ -0,0 +1,8 @@ +Content-type: text/plain + + +Date: Thu, 08 Jan 2009 17:42:29 +0000 + + +From: W. Trevor King + diff --git a/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/9b70d421-62c0-47ce-b648-93afb6a988f6/body b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/9b70d421-62c0-47ce-b648-93afb6a988f6/body new file mode 100644 index 0000000..0dfbe9d --- /dev/null +++ b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/9b70d421-62c0-47ce-b648-93afb6a988f6/body @@ -0,0 +1,37 @@ +adding a + print phys, type(phys) +to phys_to_comedi reveals: + +In [1]: import unfold, calibcant.calibrate, z_piezo_utils, numpy + +In [2]: u=unfold.unfold(controlTemp=False) +6.74763832659 +-6.74763832659 +0.0 +2.94117647059 +-2.94117647059 +0.00735294117647 +0.0 +0.0 +0.0 + +In [3]: rates = exp(numpy.linspace(log(250), log(1000), 2)) + +In [4]: for i in range(10) : + ...: for nmPsRate in rates : + ...: u.unfold(nmPsRate=nmPsRate, rel_setpoint=1, nmDist=800, sBindTime=4) + ...: u.xpWander() + ...: + ...: +setpoint = 0.850004 +approaching +0.850003814755 +0.850003814755 +0.850003814755 +0.850003814755 +None +None +--------------------------------------------------------------------------- + Traceback (most recent call last) +... + diff --git a/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/9b70d421-62c0-47ce-b648-93afb6a988f6/values b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/9b70d421-62c0-47ce-b648-93afb6a988f6/values new file mode 100644 index 0000000..2bdc66d --- /dev/null +++ b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/9b70d421-62c0-47ce-b648-93afb6a988f6/values @@ -0,0 +1,11 @@ +Content-type: text/plain + + +Date: Thu, 08 Jan 2009 17:31:28 +0000 + + +From: W. Trevor King + + +In-reply-to: a02dd6cc-132c-438a-96b2-f3677673bdfe + diff --git a/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/a02dd6cc-132c-438a-96b2-f3677673bdfe/body b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/a02dd6cc-132c-438a-96b2-f3677673bdfe/body new file mode 100644 index 0000000..f75b1b8 --- /dev/null +++ b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/a02dd6cc-132c-438a-96b2-f3677673bdfe/body @@ -0,0 +1,55 @@ +In [31]: for i in range(10) : + for nmPsRate in rates : + u.unfold(nmPsRate=nmPsRate, rel_setpoint=1, nmDist=800, sBindTime=4) + u.xpWander() + ....: + ....: +setpoint = 0.846952 +approaching +0.846952010376 +0.846952010376 +None +--------------------------------------------------------------------------- + Traceback (most recent call last) + +/thor/mroman/ in () + +/home/wking/.python/unfold.py in unfold(self, setpoint, rel_setpoint, sBindTime, nmDist, nmStep, ppStep, nmPsRate, dataDirectory, fileID) + 116 except ExceptionTooFar : + 117 try : # try for a useful surface distance... +--> 118 surfPos = self.getSurfPos(setpoint) + 119 print "Too far (surface at %g nm), stepping closer" % surfPos + 120 except z_piezo_utils.poorFit, string : # ... oh well, print what we know + +/home/wking/.python/unfold.py in getSurfPos(self, setpoint, textVerbose, plotVerbose) + 266 return out + 267 def getSurfPos(self, setpoint=2, textVerbose=False, plotVerbose=False) : +--> 268 return self.zp.pos_out2nm(z_piezo_utils.getSurfPos(self.zp, self.zp.def_V2in(setpoint), textVerbose, plotVerbose)) + 269 def curDef(self) : + 270 self.zp.updateInputs() + +/home/wking/.python/z_piezo.py in def_V2in(self, physical) + 139 def def_V2in(self, physical) : + 140 print physical, type(physical) # temporary debugging printout +--> 141 return self._jump.phys_to_out(physical) + 142 def curVals(self) : + 143 return {'Z piezo output': self.curPos(), + +/home/wking/.python/z_piezo.py in phys_to_out(self, physical) + 254 return self.AO.comedi_to_phys(0, output) + 255 def phys_to_out(self, physical) : +--> 256 return self.AO.phys_to_comedi(0, physical) + 257 + 258 class _z_piezo_ramp : + +/thor/wking/lib/python2.5/site-packages/pycomedi-0.1-py2.5.egg/pycomedi/common.py in phys_to_comedi(self, chan_index, phys) + 362 return phys + 363 def phys_to_comedi(self, chan_index, phys) : +--> 364 comedi = self._comedi.comedi_from_phys(phys, self._comedi_range[chan_index], self.maxdata[chan_index]) + 365 if self.verbose : + 366 print "%g Volts = comedi %d on subdev %d, chan %d, range [%g, %g], max %d" % (phys, comedi, self.subdev, self.chan[chan_index], self._comedi_range[chan_index].max, self._comedi_range[chan_index].min, self.maxdata[chan_index]) + +: in method 'comedi_from_phys', argument 1 of type 'double' + +In [35]: + diff --git a/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/a02dd6cc-132c-438a-96b2-f3677673bdfe/values b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/a02dd6cc-132c-438a-96b2-f3677673bdfe/values new file mode 100644 index 0000000..506afad --- /dev/null +++ b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/comments/a02dd6cc-132c-438a-96b2-f3677673bdfe/values @@ -0,0 +1,8 @@ +Content-type: text/plain + + +Date: Thu, 08 Jan 2009 17:23:49 +0000 + + +From: W. Trevor King + diff --git a/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/values b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/values new file mode 100644 index 0000000..24bff17 --- /dev/null +++ b/.be/bugs/88a58ed8-0fd9-4963-adb0-dcf6b842f2cc/values @@ -0,0 +1,17 @@ +creator: W. Trevor King + + +reporter: W. Trevor King + + +severity: minor + + +status: closed + + +summary: Crash during unfolding + + +time: Thu, 08 Jan 2009 17:23:39 +0000 + diff --git a/.be/settings b/.be/settings new file mode 100644 index 0000000..739d28c --- /dev/null +++ b/.be/settings @@ -0,0 +1,2 @@ +rcs_name: git + diff --git a/.be/version b/.be/version new file mode 100644 index 0000000..990837e --- /dev/null +++ b/.be/version @@ -0,0 +1 @@ +Bugs Everywhere Tree 1 0 diff --git a/pycomedi/simult_aio.py b/pycomedi/simult_aio.py index a497e72..0d2507c 100644 --- a/pycomedi/simult_aio.py +++ b/pycomedi/simult_aio.py @@ -324,7 +324,6 @@ values samples output on each channel are the same.""" self._comedi_internal_trigger(self.AO.subdev) self.state = "Armed" def start_read(self, in_buffer) : - print "_nsamps", self._nsamps if self.state != "Armed" : raise simAioError, "Invalid state %s" % self.state if len(in_buffer) < self._nsamps * self.AI.nchan : -- 2.26.2