Skip to content

Add XMLSIZE parameter to NDROIStat#132

Merged
gilesknap merged 1 commit intomainfrom
roistat-xml-size
Aug 29, 2025
Merged

Add XMLSIZE parameter to NDROIStat#132
gilesknap merged 1 commit intomainfrom
roistat-xml-size

Conversation

@GDYendell
Copy link
Copy Markdown
Member

I think this is supposed to be an int so I was confused why the existing XMLSIZE in the hdf5 plugin uses str

https://epics-base.github.io/epics-base/waveformRecord.html

This just adds XMLSIZE to NDROIStat for now, pending #131

@GDYendell GDYendell requested a review from gilesknap August 27, 2025 09:24
@gilesknap
Copy link
Copy Markdown
Member

Re int the auto converter cannot determine the type of dB macros (unless they have a default) so there will be a few things like that in the current support yaml.

Note that the overrides file that I linked you in slack can resolve this.

@GDYendell
Copy link
Copy Markdown
Member Author

Could we get this one merged for now so that we can progress ViSR and continue wider discussion on #133?

@gilesknap
Copy link
Copy Markdown
Member

OK - merging.

@gilesknap gilesknap merged commit 2b19a10 into main Aug 29, 2025
12 checks passed
@GDYendell GDYendell deleted the roistat-xml-size branch August 29, 2025 12:51
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