Skip to content

Process Admin Panel#327

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

Process Admin Panel#327
nickgaray merged 7 commits intoopensensorhub: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
Copy link
Copy Markdown
Member

@alexrobin alexrobin left a comment

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
Copy link
Copy Markdown

earocorn commented Apr 1, 2026

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

Process Editor (Simple)

Process Editor (GeoPTZ)

Copy link
Copy Markdown
Collaborator

@nickgaray nickgaray left a comment

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