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

Architecture v1 #1

Merged
merged 29 commits into from
Nov 13, 2024
Merged

Architecture v1 #1

merged 29 commits into from
Nov 13, 2024

Conversation

hatomist
Copy link
Collaborator

@hatomist hatomist commented Nov 9, 2024

@codekansas pls look if protobuf makes sense. incomplete but most of the essential stuff is there

Cargo.toml Outdated
edition = "2021"
license = "MIT"
repository = "https://github.com/kscalelabs/kscaleos"
description = "The K-Scale Operating System"
documentation = "https://docs.kscale.dev/kscaleos/intro"
readme = "README.md"
readme = "README.md"
Copy link
Member

Choose a reason for hiding this comment

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

vim???

(i mean, no new lines at end of file, this isn't a vscode default right?)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

LOL
no that was vscode........


// System Messages
message GetIPAddressResponse {
string ip_address = 1;
Copy link
Member

Choose a reason for hiding this comment

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

maybe multiple ip addresses

message SetCalibrationDataRequest {
string data = 1; // YAML data as string
}

Copy link
Member

Choose a reason for hiding this comment

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

add a config.proto with configuration options for rust-side

Comment on lines 70 to 74
oneof command {
PositionControl position_control = 2;
TorqueControl torque_control = 3;
VelocityControl velocity_control = 4;
}
Copy link
Member

Choose a reason for hiding this comment

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

just use this control format

{
  torque: optional[float]
  velocity: optional[float]
  position: optional[float]
  kp: optional[float]
  kd: optional[float]
}

Comment on lines 23 to 32
message ActionResult {
uint32 actuator_id = 1;
bool success = 2;
Error error = 3;
}

message ActionResponse {
bool success = 1;
Error error = 2;
}
Copy link
Member

Choose a reason for hiding this comment

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

actuator feedback

all optional except position

{
  position: float
  velocity: optional[float]
  torque: optional[float]
  temperature: optional[float]
  current_load: optional[float]
}

Copy link
Contributor

@budzianowski budzianowski left a comment

Choose a reason for hiding this comment

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

This all looks fine in principle but can you create a graph in excalidraw with connections. The architecture with the right proto will come from this straight away.

@hatomist
Copy link
Collaborator Author

hatomist commented Nov 9, 2024

  • get diagnostic logs (dmesg/services)

@hatomist
Copy link
Collaborator Author

  • proper logging
  • errors (eyre)
  • update to kos

@hatomist
Copy link
Collaborator Author

  • OTAs and diagnostics

@budzianowski budzianowski self-requested a review November 12, 2024 04:57
@@ -0,0 +1,2 @@
// TODO: Implement process manager.
// This will manage life cycle of non rust services (e.g. gstreamer, mosquitto etc)
Copy link
Contributor

Choose a reason for hiding this comment

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

Define supervisor loop called from actuator (Denys)

@hatomist hatomist marked this pull request as ready for review November 13, 2024 20:21
@hatomist
Copy link
Collaborator Author

@codekansas u ok with merging this in for ci testing? and in general

@hatomist hatomist merged commit 4bd065d into master Nov 13, 2024
1 check failed
@hatomist hatomist deleted the architecture_v1 branch November 13, 2024 20:54
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.

4 participants