Skip to content

Actor Status Monitoring & Saving to File#75

Merged
gomezzz merged 46 commits intomainfrom
monitoring
Dec 20, 2022
Merged

Actor Status Monitoring & Saving to File#75
gomezzz merged 46 commits intomainfrom
monitoring

Conversation

@gomezzz
Copy link
Collaborator

@gomezzz gomezzz commented Dec 14, 2022

Description

Summary of changes

  • Implemented OperationsMonitor to log
  • Can save logs to csv files
  • Added test for it
  • Added readme explanation of the feature

Resolved Issues

How Has This Been Tested?

  • New test

Related Pull Requests

N / A

Gabriele.Meoni and others added 18 commits December 9, 2022 14:39
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
@gomezzz gomezzz added enhancement New feature or request tests Anything related to the tests feature implementing a new feature user-facing Anything that users can interact with labels Dec 14, 2022
gomezzz and others added 2 commits December 15, 2022 11:52
Adding example on how to perform an activity.
Copy link
Collaborator

@GabrieleMeoni GabrieleMeoni left a comment

Choose a reason for hiding this comment

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

For me it's fine. However, I am not sure about the correct order for merging. Should we wait to merge this after README and ground stations? @gomezzz

@gomezzz gomezzz mentioned this pull request Dec 19, 2022
@gomezzz
Copy link
Collaborator Author

gomezzz commented Dec 19, 2022

For me it's fine. However, I am not sure about the correct order for merging. Should we wait to merge this after README and ground stations? @gomezzz

Yes after #71 and #68

@gomezzz gomezzz mentioned this pull request Dec 19, 2022
1 task
GabrieleMeoni and others added 23 commits December 20, 2022 09:56
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Fixing grammar.
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
Co-authored-by: Pablo Gómez <contact@pablo-gomez.net>
@gomezzz gomezzz merged commit 6b01df2 into main Dec 20, 2022
@gomezzz gomezzz deleted the monitoring branch December 20, 2022 19:59
@gomezzz gomezzz mentioned this pull request Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request feature implementing a new feature tests Anything related to the tests user-facing Anything that users can interact with

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Track local actor's power consumption, communicated data, performed activities, etc.

2 participants