pcdshub / pytmc

Generate EPICS IOCs and records from TwinCAT projects - along with many TwinCAT project tools
https://pcdshub.github.io/pytmc/
Other
10 stars 11 forks source link

TwinCAT does not currently allow '$' in a pragma #132

Closed klauer closed 4 years ago

klauer commented 4 years ago

With $ in the pragma, the entire pragma is ignored and not included in the .tmc.

ZLLentz commented 4 years ago

I don't have a preference for replacement symbols. Percent is fine

n-wbrown commented 4 years ago

There's a possibility this could conflict with the way arrays of objects are formatted if we're using the % symbol to specify how the array's index will be formatted in the PV name. Either way it's probably good practice to allow us to change exactly which symbol we use for this at a later date. #, @, &. and ^ could be good replacements if we need them. Maybe we start with one of the other symbols so if Beckhoff hasn't patched the $ rejection by the time we need it ('it' being the % symbol), we don't have to create a second interim convention.

klauer commented 4 years ago

Good point, @n-wbrown, I think we probably should avoid % for your stated reason...

klauer commented 4 years ago

135 will close this, using @ as the alternate character for pv: strings.