This partially reverts 1203fa9e66
LP-495 F4 USB CDC: remove internal rx_active state tracking and use actual endpoint status instead (like F1)
However rx_status is only used in the reinit case, without reintroducing issue LP-495
LP-599 Extend cloneof attribute functionality in uavobject xml
Approved-by: Vladimir Zidar <mr_w@mindnever.org>
Approved-by: Jan NIJS <dr.oblivium@gmail.com>
Approved-by: Philippe Renon <philippe_renon@yahoo.fr>
Approved-by: Lalanne Laurent <f5soh@free.fr>
LP-597 Progress bar for GCS log replay
Approved-by: Jan NIJS <dr.oblivium@gmail.com>
Approved-by: Philippe Renon <philippe_renon@yahoo.fr>
Approved-by: Lalanne Laurent <f5soh@free.fr>
- Update assessAltitude() to return the altitude in auto takeoff as well as auto land code.
Code that duplicated the calculations has been removed.
- Make pretty & update headers
Suggested by Philippe Renon:
- change the combined play-pause button to separate play and pause buttons. Hide button irrelevant button.
Code has been simplified as a result.
Fixes:
- fix situation where replay is stopped/paused and logfile is disconnected through the Disconnect button.
- remove unneeded stuff (resizing related)
- rename updateBeginAndEndTimes -> setBeginAndEndTimes
- less base height needed for the widget
- wordwrap for "Playback speed" label. Wraps when space constrains.
- set a fixed width for statusLabel to prevent jumping around when the text changes
- Update the statusLabel to "Paused" when stopped/paused.
- rename enableButtons -> enableWidgets
- clear start and end text labels when playback has finished.
- remove extraneous parentheses
- use Qt naming convention for playbackPosition -> setPlaybackPosition
- consistently use playback instead of playBack
- most places: use of "state" instead of "status"
- removed empty line
- updated file headers
- removed QThread include (leftover from debugging)
- renamed pauseAndResetPosition to pauseReplayAndResetPosition for consistency with the related functions
- Bugfix: ensure the index vectors are emptied before adding elements to them. Caused problem when replaying multiple logfiles successively.
- extra validations while indexing the logfile
- Define TIMESTAMP_SIZE_BYTES and use that instead of the hardcoded timestamp size of 4 bytes.
- modify variable case: updateBeginAndEndtimes -> updateBeginAndEndTimes
- improve qWarning messages for logfile.cpp: add the name of the function because we read the logfile in two separate functions.
- remove separate play and pause buttons, combine these functions in one playPause button.
- update icons & button texts
- move resources (icons) to logging plugin's own images folder
- explain why looking for the next timestamp after the last played one is useful.
It causes the replay process to jump over parts of the logfile where no data has been recorded
and jumps straight to the first timestamp after that part of the log where data is missing.
- Added stop button:
- start: starts from 0 position after stop signal, resumes from current position after pause signal.
- pause: freezes at current position, also freezes scopes.
- stop: stops replay and resets start position to 0 (does not close the logfile)
- Creates an index of the timestamp positions in the logfile for quick seeking to the target position in the log.
- Start, End and current position timestamps are visible in the GUI below the position slider.
- Determine replay position by moving the position slider
- Update position label while changing position bar
- Speed widget: lowest multiplier is now 0.1 instead of 0.
Only set one decimal of precision.
More decimals seem useless at this time.
- AltitudeAtState is corrected to be a positive altitude above ground.
- remove unneeded call to StabilizationDesiredGet()
- update timeout values in descriptions so they match the code
LP-303 workaround issue with full screen mode on windows
Approved-by: Philippe Renon <philippe_renon@yahoo.fr>
Approved-by: Lalanne Laurent <f5soh@free.fr>
