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

Fault Page #156

Open
RChandler234 opened this issue May 1, 2024 · 1 comment · May be fixed by #211
Open

Fault Page #156

RChandler234 opened this issue May 1, 2024 · 1 comment · May be fixed by #211
Assignees

Comments

@RChandler234
Copy link
Contributor

Description

Add a fault page. When a fault triggers it should display the last two minutes of all data points (maybe more specific) from the car

Acceptance Criteria

verify it with mock data

Proposed Solution

use callbacks when fault messages are received

Mocks

N/A

@RChandler234 RChandler234 self-assigned this May 1, 2024
RChandler234 added a commit that referenced this issue Jul 2, 2024
RChandler234 added a commit that referenced this issue Sep 22, 2024
RChandler234 added a commit that referenced this issue Sep 22, 2024
@RChandler234 RChandler234 linked a pull request Sep 22, 2024 that will close this issue
15 tasks
RChandler234 added a commit that referenced this issue Sep 24, 2024
RChandler234 added a commit that referenced this issue Sep 24, 2024
@jr1221
Copy link
Contributor

jr1221 commented Sep 26, 2024

Postgres has learned caching. However it is usually a longer term thing. Instead lets cache with scylla.

  • Cache get data by run, time, fault, etc. roputes
  • DO NOT cache car config, any other config, or the get runs or nodes endpoints. Just endpoints that deal with immutable data
  • Use something that integrates well with axum
  • Ensure max memory is eithier runtime configurable or set conservatively (under 1 gb)

Use a axum integrated caching library layer and bind only some endpoints to that layer. Configure appropriately. Avoid dedicated servers like redis, etc. Persistence not needed.

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 a pull request may close this issue.

2 participants