Skip to content

Process Admin Panel#327

Merged
nickgaray merged 7 commits into
opensensorhub:masterfrom
kalynstricklin:update-sml-process-gui
Apr 20, 2026
Merged

Process Admin Panel#327
nickgaray merged 7 commits into
opensensorhub:masterfrom
kalynstricklin:update-sml-process-gui

Conversation

@kalynstricklin

Copy link
Copy Markdown

This addition allows user to configure the process xml within the admin interface.

Comment thread sensorhub-webui-core/src/main/java/org/sensorhub/ui/SMLProcessPanel.java Outdated
Comment thread sensorhub-webui-core/src/main/java/org/sensorhub/ui/SystemIOList.java Outdated
Comment thread sensorhub-webui-core/src/main/java/org/sensorhub/ui/SystemSelectionPopup.java Outdated
Comment thread sensorhub-webui-core/src/main/java/org/sensorhub/ui/AdminUIModule.java Outdated

@alexrobin alexrobin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@kalynstricklin @nickgaray I haven't looked at all the details added to ProcessAdminPanel and ProcessFlowDiagram but since it's not used in PROD yet, I trust you guys for testing the various new features.

That said, @earocorn had made comments that are not marked as resolved so you may want to check with him and do another pass on these.

@earocorn

earocorn commented Apr 1, 2026

Copy link
Copy Markdown

@alexrobin @nickgaray Here are video examples of the process editor in action

Process Editor (Simple)

Process Editor (GeoPTZ)

@nickgaray nickgaray 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.

Make sure all files have file headers

@alexrobin

Copy link
Copy Markdown
Member

@earocorn @kalynstricklin We can merge this but did you look into the flow diagram part so we can have a graphical view of the process chain ?

@earocorn

Copy link
Copy Markdown

I looked at the flow diagram class, but it was a bit too complicated to get it to work.

I was hoping that in the new admin panel, we would have some easier-to-use JavaScript library that communicates with an admin/processing API.

Make sure all files have file headers

Also, I've added license headers to the new files.

@nickgaray nickgaray merged commit 16f3b5c into opensensorhub:master Apr 20, 2026
1 check passed
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.

4 participants