corrects how each channel should be cleared

.gitea/workflows/action.yaml

@@ -0,0 +1,27 @@
+name: Example Action
+on: [push]
+
+jobs:
+  Lint:
+    runs-on: linepics
+    steps:
+      - name: checkout repo
+        uses: actions/checkout@v4
+      - name: cppcheck
+        run: cppcheck --std=c++17 --addon=cert --addon=misc --error-exitcode=1 src/*.cpp
+      - name: formatting
+        run: clang-format --style=file --Werror --dry-run src/*.cpp
+  BuildAndTest:
+    runs-on: linepics
+    steps:
+      - name: checkout repo
+        uses: actions/checkout@v4
+      - name: build
+        run: |
+          sed -i 's/ARCH_FILTER=.*/ARCH_FILTER=linux%/' Makefile
+          echo "LIBVERSION=${CI_COMMIT_TAG:-0.0.1}" >> Makefile
+          make install
+      - name: test
+        run: |
+          cd test
+          python3 test.py

.gitlab-ci.yml

@@ -1,61 +0,0 @@
-default:
-  image: docker.psi.ch:5000/sinqdev/sinqepics:latest
-    
-stages:
-  - lint
-  - build
-  - test
-
-cppcheck:
-  stage: lint
-  script:
-    - cppcheck --std=c++17 --addon=cert --addon=misc --error-exitcode=1 src/*.cpp
-  artifacts:
-    expire_in: 1 week
-  tags:
-    - sinq
-
-formatting:
-  stage: lint
-  script:
-    - clang-format --style=file --Werror --dry-run src/*.cpp
-  artifacts:
-    expire_in: 1 week
-  tags:
-    - sinq
-
-# clangtidy:
-#   stage: lint
-#   script:
-#     - curl https://docker.psi.ch:5000/v2/_catalog
-# #     - dnf update -y
-# #     - dnf install -y clang-tools-extra
-# #     - clang-tidy sinqEPICSApp/src/*.cpp sinqEPICSApp/src/*.c sinqEPICSApp/src/*.h -checks=cppcoreguidelines-*,cert-*
-# #   tags:
-# #     - sinq
-
-build_module:
-  stage: build
-  script:
-    - sed -i 's/ARCH_FILTER=.*/ARCH_FILTER=linux%/' Makefile
-    - echo "LIBVERSION=${CI_COMMIT_TAG:-0.0.1}" >> Makefile
-    - make install
-    - cp -rT "/ioc/modules/counterbox/$(ls -U /ioc/modules/counterbox/ | head -1)" "./counterbox-${CI_COMMIT_TAG:-$CI_COMMIT_SHORT_SHA}"
-  artifacts:
-    name: "counterbox-${CI_COMMIT_TAG:-$CI_COMMIT_SHORT_SHA}"
-    paths:
-      - "counterbox-${CI_COMMIT_TAG:-$CI_COMMIT_SHORT_SHA}/*"
-    expire_in: 1 week
-  when: always
-  tags:
-    - sinq
-
-test_module:
-  stage: test
-  script:
-    - mkdir -p "/ioc/modules/counterbox"
-    - cp -rT "./counterbox-${CI_COMMIT_TAG:-$CI_COMMIT_SHORT_SHA}" "/ioc/modules/counterbox/${CI_COMMIT_TAG:-0.0.1}"
-    - python3 test/test.py
-  when: always
-  tags:
-    - sinq

db/channels.db

@@ -40,17 +40,51 @@ record(bi, "$(INSTR)$(NAME):S$(CHANNEL)")
 ################################################################################
 # Count Commands
 
+# Unfortunately, clearing the channels is somewhat complicated as a result of
+# the addition of more channels over time and minimal changes to the underlying interface
+#
+# Urs Greuter provided the following explanation:
+#
+# bei den Befehlen CC r und HC r ist der Parameter r als bit-Maske zu verstehen:
+#
+# Bit0: Zähler Channel 1
+# Bit2: Zähler Channel 2
+# Bit3: Zähler Channel 3
+# Bit4: Zähler Channel 4
+# Bit5: Zähler Channel Timer
+# Bit6: Zähler Channel 5
+# Bit7: Zähler Channel 6
+# Bit8: Zähler Channel 7
+# Bit9: Zähler Channel 8
+#
+# Beispiele:
+# CC 1 setzt den Zähler des Channels 1 zurück
+# CC 4 setzt den Zähler des Channels 3 zurück
+# CC 5 setzt gleichzeitig die Zähler der Channels 1 und 3 zurück
+# CC 16 ist gleichbedeutend wie CT (Timer zurücksetzen)
+# CC 511 setzt gleichzeitig die Zähler aller Kanäle (auch des Timers) zurück.
+
+record(calc, "$(INSTR)$(NAME):BM$(CHANNEL)")
+{
+ field(DESC, "Bit Mask for Channel")
+ field(INPA, $(CHANNEL))
+ field(CALC, "A > 4 ? 2 ^ A : 2 ^ (A-1)")
+ field(PINI, "YES")
+}
+
 record(longout, "$(INSTR)$(NAME):C$(CHANNEL)")
 {
  field(DESC, "Clear the current channel count")
  field(DTYP, "stream")
- field(OUT, "@$(PROTO) clearChannel($(INSTR)$(NAME):, $(CHANNEL)) $(ASYN_PORT)")
+ field(OMSL, "closed_loop")
+ field(DOL, "$(INSTR)$(NAME):BM$(CHANNEL) NPP")
+ field(OUT, "@$(PROTO) clearChannel($(INSTR)$(NAME):) $(ASYN_PORT)")
  field(FLNK, "$(INSTR)$(NAME):T$(CHANNEL)")
 }
 
 record(ao,"$(INSTR)$(NAME):THRESH$(CHANNEL)")
 {
- field(DESC, "Sets minimum rate for counting to proceed")
+ field(DESC, "Sets min rate for counting to proceed")
  field(OMSL, "supervisory")
  field(OROC, "0")
  field(OUT, "@$(PROTO) setMinRate($(INSTR)$(NAME):, $(CHANNEL)) $(ASYN_PORT)")

db/daq.proto

@@ -5,17 +5,18 @@ OutTerminator = CR;
 InTerminator  = CR;
 ReadTimeout    = 100;
 WriteTimeout   = 100;
-ReplyTimeout   = 200;
+ReplyTimeout   = 1000;
-LockTimeout    = 450;
+LockTimeout    = 500;
 
 initialise {
  out "RMT 1"; # Turn on Remote Control
  in;
  out "ECHO 2"; # Ask for reponses
  in "%(\$1MsgTxt)s"; # Clear MsgTxt on Init
- @mismatch{
+ # Probably should cause the ioc to just exit
-  exec 'echo "Failed to configure DAQ" && exit(1)';
+ # @mismatch{
- }
+ #  exec 'echo "Failed to configure DAQ" && exit(1)';
+ # }
 }
 
 fullReset {
@@ -74,39 +75,25 @@ clearTimer{
 }
 
 clearChannel{
- out "CC \$2";
+ out "CC %d";
  in;
  @mismatch{in "%(\$1MsgTxt)s";}
 }
 
 clearCounter4 {
- out "CC 1";
+ out "15";
- in;
- out "CC 2";
- in;
- out "CC 3";
- in;
- out "CC 4";
  in;
  @mismatch{in "%(\$1MsgTxt)s";}
 }
 
 clearCounter8 {
- out "CC 5";
+ out "CC 480";
- in;
- out "CC 6";
- in;
- out "CC 7";
- in;
- out "CC 8";
  in;
  @mismatch{in "%(\$1MsgTxt)s";}
 }
 
 clearCounter10 {
- out "CC 9";
+ out "CC 1536";
- in;
- out "CC 10";
  in;
  @mismatch{in "%(\$1MsgTxt)s";}
 }

sim/daq_sim.py

@@ -35,7 +35,7 @@ class DAQ:
         self.monitor = 0
 
         self.minratechannel = 0
-        self.minrates = [2] * self.total_channels
+        self.minrates = [2] * (self.total_channels + 1)
 
         self.gate_config = [
            #(enabled, count high/low),
@@ -92,7 +92,7 @@ class DAQ:
         if self.countmode == 'time':
             if elapsed < self.presettime:
 
-                if self.minratechannel >= 0 and self.rates[self.minratechannel] < self.minrates[self.minratechannel]:
+                if self.minratechannel > 0 and self.rates[self.minratechannel - 1] < self.minrates[self.minratechannel]:
                     # adjust the starttime, so that it is as if this polling period didn't happen
                     self.starttime += elapsed - self.elapsed
                     self.status = 5
@@ -115,7 +115,7 @@ class DAQ:
         elif self.countmode == 'count':
             if self.getMonitorCount() < self.presetcount:
 
-                if self.minratechannel >= 0 and self.rates[self.minratechannel] < self.minrates[self.minratechannel]:
+                if self.minratechannel > 0 and self.rates[self.minratechannel - 1] < self.minrates[self.minratechannel]:
                     # adjust the starttime, so that it is as if this polling period didn't happen
                     self.starttime += elapsed - self.elapsed
                     self.status = 5
@@ -171,16 +171,16 @@ class DAQ:
         return float(self.rates[channel - 1])
 
     def getMinRateChannel(self):
-        return self.minratechannel + 1
+        return self.minratechannel
 
     def setMinRateChannel(self, channel):
-        self.minratechannel = channel - 1
+        self.minratechannel = channel
 
     def getMinRate(self, channel):
-        return self.minrates[channel - 1]
+        return self.minrates[channel]
 
     def setMinRate(self, channel, rate):
-        self.minrates[channel - 1] = rate
+        self.minrates[channel] = rate
 
     def getGateStatus(self, channel):
         return self.gate_config[channel - 1]

test/st.cmd

@@ -6,9 +6,8 @@ require sinqDAQ
 
 epicsEnvSet("STREAM_PROTOCOL_PATH","./db")
 epicsEnvSet("INSTR","SQ:TEST:")
-epicsEnvSet("NAME","CB_TEST")
 
 epicsEnvSet("SET_SIM_MODE","") # Run Simulation Instead of Actual Interface
-runScript "$(sinqDAQ_DIR)daq_2nd_gen.cmd" "DAQ_IP=127.0.0.1, DAQ_PORT=2000"
+runScript "$(sinqDAQ_DIR)daq_2nd_gen.cmd" "NAME=CB_TEST, DAQ_IP=127.0.0.1, DAQ_PORT=2000"
 
 iocInit()