DUNE-DAQ / nanorc

2 stars 2 forks source link

bug fix for expert command, include and exclude #219

Closed LJoyL closed 10 months ago

LJoyL commented 10 months ago

PR for issue #218

plasorak commented 10 months ago

Do include/exclude require the same treatment too?

LJoyL commented 10 months ago

I supposed it was interesting if it end up being implemented in an integration test in the future. I only need the expert command in my case so we can revert for include and exclude if it is not required.

plasorak commented 10 months ago

Hi Leo, do you have an example of an expert command json?

plasorak commented 10 months ago

ok, I found it:

{
    "id": "record",
    "entry_state": "ANY", # can be omitted
    "exit_state": "ANY", # can be omitted
    "data": {} # can be omitted
}