UbiquityRobotics / pi_image_build

GNU General Public License v2.0
16 stars 6 forks source link

Login message for incorrect system time. #7

Open rohbotics opened 4 years ago

rohbotics commented 4 years ago

The URL is made up right now, we should have a page for RTCs that is relevant to both Magni and non-Magni image users.

Fixes: https://github.com/UbiquityRobotics/pi_image_build/issues/6

rohbotics commented 4 years ago

Yes the battery installation is covered there, but I thought it would be useful to have a page that would be applicable to non-Magni image users as well, because they are going to get this message almost every time they boot up, and they can't exactly install a battery on their non-existent MCB

mjstn commented 4 years ago

I would rather point them to a generic page somebody (must have) written about rviz and time sync. That way it decouples us from a complex support issue. We would have to suggest something like that Dallas semi plug in part and drivers and blah blah blah. I don't know how far we have to bend over backwards. Best to point them to some ROS Rviz page that already discusses the fixes. So yes, 'nice idea' but sort of above and beyond. Is there anybody complaining about that right now (to us)?

mjstn commented 4 years ago

If I find something it would go to this page which I created to help non-magni users. I do care Rohan, it is just a question of how valuable it would be to solve this for them if we could just point them to some external page about it. I will look for such a page and get back to you.

rohbotics commented 4 years ago

I don't think we need to really talk about the rviz issue at all, all I was thinking about was a page that says that keeping time is important, and that if they are using a Magni we have solved this problem they need to install a battery (insert link here). If they are not using Magni, they can get one of these RTC modules (list of links) and configure it by doing XYZ simple step.

If we add this warning and don't point to something relevant for both Magni and non-Magni users, I will probably get at least 4 emails a week asking about it, so I would rather come out ahead and provide a link. A vast majority of our image users don't have a Magni, so I want to make sure that we have something reasonable there.

rohbotics commented 3 years ago

@mjstn @MoffKalast Updated the URL so now it actually works. Let me know if this is still good to merge.

MoffKalast commented 2 years ago

We should probably integrate this into pi_image2