Skip to content

Conversation

jnavno
Copy link

@jnavno jnavno commented Jul 16, 2025

What does this PR do?

Adds a new TreeGuardEvent message to capture battery voltage, SOC and classification results from our Tree‑Guard sensor module.

Checklist before merging

  • All top level messages commented
  • All enum members have unique descriptions

// TreeGuardEvent carries a snapshot of battery readings plus your
// vibration‑classification result.
message TreeGuardEvent {
// Battery voltage in volts.
float battery_voltage = 1;

// State‑of‑charge percentage (0.0–100.0).
float battery_soc = 2;

// The detected event type.
EventType event = 3;
}

enum EventType {
// Event could not be classified.
UNCLASSIFIED = 0;
// Likely machete impact detected.
LIKELY_MACHETE = 1;

}

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@thebentern
Copy link
Contributor

@jnavno can you tell us a little bit about this use case? Is it for detecting illegal logging activities in protected forests? Looks really interesting

@jnavno jnavno closed this Jul 17, 2025
@jnavno
Copy link
Author

jnavno commented Jul 17, 2025

Thank you for reviewing this. I've realized this pull request was opened prematurely and contains work that isn't ready for public contribution yet. I apologize for any inconvenience.

@jnavno
Copy link
Author

jnavno commented Jul 17, 2025

@jnavno can you tell us a little bit about this use case? Is it for detecting illegal logging activities in protected forests? Looks really interesting

Thanks for your interest and question! I've closed the pull request because the content wasn't ready for public consumption or discussion. I'm not able to discuss the details further at this time, but I appreciate your understanding.

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.

3 participants