-
Notifications
You must be signed in to change notification settings - Fork 1.7k
doco/website: Add footnote to VW-group cars to prevent user confusion about where harness connects (option 2 - brand-specific footnote) #2842
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
base: master
Are you sure you want to change the base?
Conversation
… about where harness connects (option 2 - brand-specific footnote) Follow-up from commaai/openpilot#36325 Related alternative PR: commaai#2841 This brand-specifc copy option creates more footnotes in CARS.md, but gives the customer more guidance about where to plug the harness. Thoughts?
60b4893 to
097340a
Compare
|
a good idea! thought of sending it back because of this. limited in tools, I am not able to install it currently. already spent time on this. honestly I wouldn't have bought it in the first place if I knew that beforehand. definitely needs to be added on the homepage |
|
@sshane @adeebshihadeh From a product owner lens, are you happy to merge this in and propagate the resulting CARS.md into the openpilot repo and then (the ultimate goal) to the website repo, so that the website has the additional footnote? Note there's a 2nd variant of this PR that has a more generic implementation that could be used for multiple brands (but we'd need a more generic text) #2841 I think either would improve the customer experience a lot and avoid surprises and confusion. (Note: I haven't actually built the CARS.md as I don't know the process - being a Python newb - but static analysis seems happy, so hopefully the code will work as is) |
|
We have a few harnesses like this, if you think this disproportionally affects VW for some reason, we can merge this now. Just needs some clean up |
|
@sshane It keeps coming up in the VW Discord so that's where I've encountered it, after my own experience (see also Speedygonzales' comment above). Regarding your comment in the generic PR about adding a video, I agree and this could be in addition to this PR. There's a great install video by Konik on Youtube we could use for VW cars: https://www.youtube.com/watch?v=__RwCHBcLVg (it shows the J533 module really well but also shows a non-Comma harness so not sure how sensitive you are to that. I previously mentioned this idea in my comment a month ago https://discordapp.com/channels/469524606043160576/534359517836607488/1427237094782140507 ). I could add the video to this PR (if that's the pattern you prefer, i.e. brand-specific) together with the merge conflicts resolution if you're happy to have the video. Or if you prefer the generic solution, I could repurpose this PR to just add the video which would already independently help with mitigating the confusion. |
jyoung8607
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with @borhub that it's a problem, it comes up constantly. I also agree with @sshane that it's not a problem exclusive to VW. We need to Do Something.
A footnote may be part of the solution, but I don't know how many users will notice that. I suspect they're just going to google "how to install comma [3/3X/4]" and they're going to find people removing the camera cover.
I feel like the solution might be a brand-agnostic footnote for cars with custom install procedures AND a physical label stuck to those harnesses that links to the relevant custom install guide. Something like "read me first!" and a QR code.
Follow-up from commaai/openpilot#36325
The Comma website is missing information for VW car users informing them that the J533 harness doesn't connect to the usual location on the windshield camera as advertised on the Comma website. This omission caused users to be confused and their Comma installation delayed when the harness connector didn't match the camera connector which they expected to plug into. People think they got the wrong connector, have to go to Discord etc.
This change fixes the customer experience by adding a footnote that directs the users to connect to the correct module under the dash.
The brand-specific copy solution option here leads to creation of more footnotes in CARS.md as more footnotes are added into respective brands'
values.pyfiles, but it gives the customer a clearer guidance about where to plug the harness, likely reducing some Discord back-and-forths, consumers sending devices back etc. The CARS.md would end up with one footnote for each brand, but this might be a good tradeoff for getting greater end-customer clarity on the website.The brand-agnostic, more generic copy alternative is in PR #2841