Skip to content

Improvements for GUI #10

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
viacheslauK merged 49 commits into from
Feb 4, 2026
Merged

Improvements for GUI #10

viacheslauK merged 49 commits into from
Feb 4, 2026

Conversation

@viacheslauK
Copy link
Collaborator

@viacheslauK viacheslauK commented Jan 30, 2026 •
edited
Loading

Brief:

  • reworking of FBs properties;
  • reworking of FBs statuses;
  • merging two FB - Raw and JSON to MQTTSubscriberFb;
  • preview signals for publisher and subscriber FBs;
  • arrays for JSON decoder FB;
  • arrays for Single mode in publisher FB;
  • common improvements and bugfixing;

@viacheslauK
mqtt: renaming
0eec736
@viacheslauK
mqtt: componentStatus error for PublisherFB if connection lost
2260aeb
@viacheslauK
mqtt: warning status for a publisher if there are no connected input …
55dae62 
…ports
@viacheslauK
mqtt: warning for a publisher FB if the topic property is empty
3d599bd
@viacheslauK
mqtt: removing SignalName property from a Publisher FB
da6eff2
@viacheslauK
mqtt: merging two FB - Raw and JSON to MQTTSubscriberFb
835c248
@viacheslauK
mqtt: Topics property for publisherFB
bb3b570
@viacheslauK
mqtt: renaming; removing unused code
8d90647
@viacheslauK
mqtt: SignalValueJSONKey property for publisher FB
821c758
@viacheslauK
mqtt: EnablePreviewSignal for subscriber FB
19dc87d
@viacheslauK
mqtt: EnablePreviewSignal for publisher FB
71cf5c3
@viacheslauK
mqtt: Schema property for publisher FB
3b67aa3
@viacheslauK
mqtt: renaming, README updating
828314e
@viacheslauK
mqtt: status helper - check if status has been set
0172d86
@viacheslauK
mqtt: MqttPublisherFbImpl::readProperties fix
acfb7de
@viacheslauK
mqtt: fix for subscriber component status
024aed4
@viacheslauK
mqtt: memory leak fix
d7a884f
@viacheslauK
mqtt: publisher test
2be52e5
@viacheslauK
mqtt: SharedTs property has been removed from publisher; removing of …
5d9f6b3 
…SignalArrayAtomicSampleHandler handler
@viacheslauK
mqtt: removed() for publisher FB
445dd39
@viacheslauK
mqtt: parent for all publisher handlers; propper addInputPots for mul…
3b7ce49 
…tiReader
@viacheslauK
mqtt: new checks for publisher FB signal validation
627a484
@viacheslauK
mqtt: publisher FB statuses improvement
e784024
@viacheslauK
gitignore
0512e59
@viacheslauK
mqtt: removing SubscribingStatus from subscriber FB
eb867a4
@viacheslauK
mqtt: removing parsing status from JSON decoder FB; updating parsing …
ec7f92e 
…error messages
@viacheslauK
mqtt: publisher FB deadlock fix
8aba451
@viacheslauK
mqtt: array parsing for decoder FB
6e413ea
@viacheslauK
mqtt: publisher fix - processing SampleCount() to extract all samples…
ebf5c01 
… from DataPacket
@viacheslauK
mqtt: new publisher mode with shared ts arrays
4c740d2
@viacheslauK viacheslauK self-assigned this Jan 30, 2026
@viacheslauK viacheslauK marked this pull request as ready for review January 30, 2026 17:03
@viacheslauK viacheslauK force-pushed the improvements-for-gui branch 2 times, most recently from 5ffeefc to 5c4c297 Compare February 3, 2026 11:07
NikolaiShipilov
NikolaiShipilov approved these changes Feb 4, 2026
View reviewed changes
Copy link

@NikolaiShipilov NikolaiShipilov left a comment

Choose a reason for hiding this comment

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

lgtm !

@viacheslauK viacheslauK merged commit 5d4e584 into main Feb 4, 2026
2 checks passed
@viacheslauK viacheslauK deleted the improvements-for-gui branch February 4, 2026 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NikolaiShipilov NikolaiShipilov NikolaiShipilov approved these changes

Assignees

@viacheslauK viacheslauK

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@viacheslauK @NikolaiShipilov