Skip to content

Conversation

@mhasself
Copy link
Member

WIP @merny93

Examples session_data for advertise:

{
  "settables": {
    "global_options": {
      "restart": []
    },
    "rtd_0": {
      "logdac": [
        "int",
        [
          0,
          15
        ]
      ],
      "something": [
        "float",
        [
          0,
          15.1
        ]
      ],
      "uvolts": [
        "bool",
        []
      ],
      "choosable": [
        "enum",
        {
          "a": 1,
          "b": 2
        }
      ]
    }
  },
  "last_update": 1765818866.1055603
}

@merny93
Copy link

merny93 commented Dec 15, 2025

This all looks great!

I tried to add a way to specify the file path of the excitation config as discussed.
I also added a "Crate status" monitor but the OpReading type may not be best suited for the log messages that it displays. Is there some scrollable or expandable reading type?

Should osc-web components be added for all "user facing" commands such that we can hide the generic commands?

@mhasself
Copy link
Member Author

I tried to add a way to specify the file path of the excitation config as discussed.

Good ... just note "upload" is confusing since it won't be possible to upload from the client; I get that you're uploading from the server to the device though.

I also added a "Crate status" monitor but the OpReading type may not be best suited for the log messages that it displays. Is there some scrollable or expandable reading type?

Not at present. There's a big text modal dialog thing I use for the session status display but I think something more embedded would be better for this. I can try to cobble one if you don't get to it.

Should osc-web components be added for all "user facing" commands such that we can hide the generic commands?

It's up to you -- some panels provide quite a bit of specialized control (e.g. ACU panel) while others provide none. My advice would just be to not "hide" too much stuff unless truly all functionality has been made available through the specialized controls.

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.

3 participants