Skip to content

Added the alarmDemo IOC as an example app. - #64

Open
pheest wants to merge 1 commit into
epics-modules:masterfrom
pheest:Include_alarmDemo
Open

pheest wants to merge 1 commit into
epics-modules:masterfrom
pheest:Include_alarmDemo

Conversation

@pheest

@pheest pheest commented Oct 28, 2026

Copy link
Copy Markdown
Collaborator

It can be run with or without PINI, RVAL (for suitable types) and VAL initialisation.

It can be run with or without PINI, RVAL (for suitable types) and VAL initialisation.

@tynanford tynanford left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @pheest this looks good. Just 2 small comments

py "alarm.build()"

## Load record instances
dbLoadRecords("db/alarm.db","PINI=${PINI}, RVAL=${RVAL}, HASH=${HASH}")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

HASH should be HASHTAG

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Please could you explain why you feel this is important?
I personally prefer the (shorter) HASH.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

HASH is fine with me but the db file is using HASHTAG still for the macro name

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

just a copy and paste leftover from the original demo repo i think

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oh! That is a mistake. I intended HASH throughout.

@@ -0,0 +1,23 @@
#!../../bin/linux-x86_64/softIocPy3.6

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

we should make st.cmd and st.sh executable

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I thought I had done. 'Executable' is of course not a Windows attribute, but the property is settable by Git for Windows and this has worked for me in the past before making commits.

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants