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

Implement file uploads #51

Open
3 of 5 tasks
adeebshihadeh opened this issue Jul 5, 2024 · 8 comments · Fixed by #216
Open
3 of 5 tasks

Implement file uploads #51

adeebshihadeh opened this issue Jul 5, 2024 · 8 comments · Fixed by #216
Labels
bounty comma.ai/bounties enhancement new feature or request
Milestone

Comments

@adeebshihadeh
Copy link
Contributor

adeebshihadeh commented Jul 5, 2024

This is what connect looks like currently:
image

Instead, I think we want a table of segments and all the file types: qlog, qcam, rlog, fcam, etc.

Either need a comma 3/3X to test this, or jury-rig connect to talk to an athena instance on your PC.

Requirements:

  • Upload files for route
  • Upload files for segment @incognitojam working on it
  • View upload queue
  • Offline queue
  • Cancel upload/clear queue
@adeebshihadeh adeebshihadeh added bounty comma.ai/bounties enhancement new feature or request labels Jul 5, 2024
@adeebshihadeh adeebshihadeh added this to the 1.0 milestone Jul 11, 2024
@DGoiana
Copy link

DGoiana commented Aug 23, 2024

Why doesn't the build look like that?

@LauraCastrillonMp
Copy link

Where in the build is this functionality? Because it doesn't look like the one in the image

@incognitojam
Copy link
Member

It looks like that in https://github.com/commaai/connect, the task is to create a replacement for new-connect

@helperfunc
Copy link

image The demo from https://connect.comma.ai/1d3dc3e03047b0c7/000000dd--455f14369d shows device offline, and didn't show upload. How can I know what to fix for this ticket?

@mfeinLearn
Copy link

mfeinLearn commented Jan 15, 2025

Hey! Could you clarify what you mean by a table of segments? Are you referring to a structured representation of the routes, along with the different file types available for download? @incognitojam @adeebshihadeh

@JacksonRT
Copy link

@mfeinLearn I believe that they want a organized grid like layout so you can upload specific parts of segments biased on their types of engagements and file types? Kind of like a file gallery.

This was referenced Mar 22, 2025
incognitojam pushed a commit that referenced this issue Mar 27, 2025
last part of #51

handles device disconnected, no items in queue, and rendering a list of
files being uploaded

possible features to add in the future:
- nice animations
- displaying the offline queue
- simplify the polling loop with something like
https://tanstack.com/query/latest/docs/framework/solid/overview
@greatgitsby
Copy link
Contributor

spoke on discord with @incognitojam, implementing the offline queue will complete the requirements of this issue

@incognitojam incognitojam moved this from Open to Locked in openpilot bounties Mar 27, 2025
@incognitojam
Copy link
Member

Locked to @greatgitsby (should have done this sooner)

incognitojam pushed a commit that referenced this issue Mar 28, 2025
#51

tested locally:
1. when device offline, upload request puts items in offline queue,
showing them in the queue in the UI with a "Queued" message
2. when device comes online, status switches to "Waiting" until Athena
server pushes the queued requests
3. finally, messages start coming in and processing as normal
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bounty comma.ai/bounties enhancement new feature or request
Projects
Status: TODO
Status: Locked
Development

Successfully merging a pull request may close this issue.

8 participants