Skip to content
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.

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

Add log view #253

Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

Add log view #253

wants to merge 31 commits into from

Conversation

Johannes-Thiel
Copy link
Contributor

@Johannes-Thiel Johannes-Thiel commented Jan 13, 2025

With this PR the rosys.notify were reviewed and fixed. The Log view was added to the /dev page. This showes up to 100 past notifications.

@Johannes-Thiel Johannes-Thiel added this to the 0.8.0 milestone Jan 13, 2025
@pascalzauberzeug pascalzauberzeug changed the title adding Log view Add Log view Jan 15, 2025
@pascalzauberzeug pascalzauberzeug added the enhancement New feature or request label Jan 15, 2025
Johannes-Thiel and others added 14 commits January 15, 2025 15:50
while migrating the configs the config for f10 was lost. This is a
restoration of the config
For now I just copied the config of F18.
For F19 the weeding tool needs to be removed

---------

Co-authored-by: Johannes-Thiel <[email protected]>
This change allows to see the current state of the navigation and of the
automation. This is shown as a mermaid diagramm.
We will add GNSS code to Rosys in
zauberzeug/rosys#231 and this PR implements it
for this repo.

ToDos:
- [x] Add Kalman Filter from
#198
- [ ] Wait for [Rosys
0.22.0](https://github.com/zauberzeug/rosys/milestones) with
zauberzeug/rosys#242

Migration Guide:

1. In your Septentrio GNSS Module change the NMEA/SBF OUT messages from
GGA+GNS+HDT to GGA+GST+HRP.
2. With the field friend container shut down, rename the extrinsics
frame_id of the camera calibration from _rosys.odometer.prediction_ to
_field_friend.robot_locator_
(~/.rosys/field_friend.vision.calibratable_usb_camera_provider.json)

---------

Co-authored-by: Falko Schindler <[email protected]>
When running an automation we want to make sure the detector is running.
If the detector is not sending detections for one second, there will be
a warning.
We changed the f14 and f15 back to rb37 and rb39. To be consistent the
config name was changed
Removing the old obsolete camera config file. These configs can be found
in the camera config files of the individual robots.
This change allows to see the current state of the navigation and of the
automation. This is shown as a mermaid diagramm.
We will add GNSS code to Rosys in
zauberzeug/rosys#231 and this PR implements it
for this repo.

ToDos:
- [x] Add Kalman Filter from
#198
- [ ] Wait for [Rosys
0.22.0](https://github.com/zauberzeug/rosys/milestones) with
zauberzeug/rosys#242

Migration Guide:

1. In your Septentrio GNSS Module change the NMEA/SBF OUT messages from
GGA+GNS+HDT to GGA+GST+HRP.
2. With the field friend container shut down, rename the extrinsics
frame_id of the camera calibration from _rosys.odometer.prediction_ to
_field_friend.robot_locator_
(~/.rosys/field_friend.vision.calibratable_usb_camera_provider.json)

---------

Co-authored-by: Falko Schindler <[email protected]>
field_friend/automations/implements/tornado.py Outdated Show resolved Hide resolved
field_friend/automations/navigation/navigation.py Outdated Show resolved Hide resolved
field_friend/automations/puncher.py Outdated Show resolved Hide resolved
field_friend/hardware/external_mower.py Outdated Show resolved Hide resolved
field_friend/log_configuration.py Show resolved Hide resolved
@LukasBaecker LukasBaecker changed the title Add Log view Add log view Jan 24, 2025
@Johannes-Thiel Johannes-Thiel marked this pull request as ready for review January 24, 2025 07:40
Copy link
Contributor

@pascalzauberzeug pascalzauberzeug left a comment

Choose a reason for hiding this comment

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

@Johannes-Thiel I made some changes to bring it into line with the other new development ui elements, since we're trying to move away from the different blue blocks.

  • Combine LogView and LogMonitor to one class with a ui function
  • Give it a fixed size and its own row to avoid it jumping around

You can merge if you're okay with the changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants