Skip to content

Add DAS240 IOC#955

Draft
FreddieAkeroyd wants to merge 2 commits into
masterfrom
add_DAS240
Draft

Add DAS240 IOC#955
FreddieAkeroyd wants to merge 2 commits into
masterfrom
add_DAS240

Conversation

@FreddieAkeroyd
Copy link
Copy Markdown
Member

@FreddieAkeroyd FreddieAkeroyd commented Mar 17, 2026

Description of work

Create IOC for Sefram DAS240

To test

See ISISComputingGroup/IBEX#8934

Acceptance criteria

List the acceptance criteria for the PR

  • ** If this PR changes GALIL / GALILMUL ioc are these changes applicable to both the old (galil-old branch based) and new (master branch based) drivers? If so have all appropriate PRs been created **

Code Review

Functional Tests

  • IOC responds correctly in:
    • Devsim mode
    • Recsim mode
    • Real device, if available
  • Supplementary IOCs (..._0n where n>1) run correctly
  • Log files do not report undefined macros (serach for macLib: macro to find instances of macLib: macro [macro name] is undefined...

Final steps

  • Update the IOC submodule in the main EPICS repo. See Git workflow page for details.
  • Reviewer has merged the associated PR for the release notes

$(APPNAME)_DBD += drvAsynSerialPort.dbd
$(APPNAME)_DBD += drvAsynIPPort.dbd
$(APPNAME)_DBD += luaSupport.dbd
$(APPNAME)_DBD += stream.dbd
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this actually use stream device?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and any other dbds loaded in here - i feel we should cut down if we're not using them as it means other facilities will need them too

Comment on lines +46 to +47
## Start any sequence programs
#seq sncxxx,"user=faa59"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not needed?

## Analog data: up to 200 acquisition data channels
## data channels are added in blocks of 20, so just do 20 channels for now (which is 40 16 bit words for 32bit float)
## modbus function code 4 for read input register
drvModbusAsynConfigure("ACQ1", "$(DEVICE)", 0, 4, 8, 40, 0, 1000, "DAS240")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could we have the signature of this func in a comment above? i find it a bit annoying to have to go and search for this in the support module in eg. a support call otherwise, particularly for things with loads of args like this

Comment thread DAS240/.gitignore
@@ -0,0 +1,40 @@
# SPDX-FileCopyrightText: 2022 Argonne National Laboratory
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you actually have per-directory git ignores?

Comment thread DAS240/README.md
@@ -0,0 +1 @@
IOC for Sefram DAS240 No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

local readme doesn't seem that needed here either

Comment thread DAS240/configure/RELEASE
SNCSEQ=$(SUPPORT)/seq/master
SQLITE=$(SUPPORT)/sqlite/master
SSCAN=$(SUPPORT)/sscan/master
STREAMDEVICE=$(SUPPORT)/StreamDevice/master
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as per comment on dbds we're not using

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants