-
Notifications
You must be signed in to change notification settings - Fork 169
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
Dont send targets or cal images to UI #1200
Conversation
Closes #1198 |
photon-core/src/main/java/org/photonvision/common/configuration/PhotonConfiguration.java
Show resolved
Hide resolved
...on-core/src/main/java/org/photonvision/vision/calibration/CameraCalibrationCoefficients.java
Outdated
Show resolved
Hide resolved
Run lint Update UICameraCalibrationCoefficients.java Update RequestHandler.java asdf fdas
04aa5af
to
6633496
Compare
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.
On systems with slow internet (FRC bandwidth limited radio) there are 1-3+ seconds of waiting for images to load which isn't ideal as people may think something isn't working. While the images are being fetched from the backend, the UI should show a loading symbol and a message as a placeholder until the fetch is complete and the correct image is shown.
Reduces bandwidth requirements by being much lazier about how much calibration data is sent to the UI.
Trying to export cal or settings over robot wifi still sucks and is super slow. I think a modal to show this during calibration export makes sense?
also adds warnings on export time
![image](https://private-user-images.githubusercontent.com/29715865/300186742-181a5343-614d-4ac4-ad82-d46737b388af.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjExMTM0NDQsIm5iZiI6MTcyMTExMzE0NCwicGF0aCI6Ii8yOTcxNTg2NS8zMDAxODY3NDItMTgxYTUzNDMtNjE0ZC00YWM0LWFkODItZDQ2NzM3YjM4OGFmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzE2VDA2NTkwNFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTE0YjY1NDcyMGI0NTI1NjZlZTVmYzEwM2Y0YmZkZWFlNzI4MDliMWJhZGU4MDVkMDA5MjIwYjc4YjY2OWRmYmImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.jVTIBqTi9NOrVI8JGwS8_PqOCKpNYqveSomN9l4nPyU)
![image](https://private-user-images.githubusercontent.com/29715865/300186747-01d8f7db-d677-4b6b-bd39-a8b8fa219446.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjExMTM0NDQsIm5iZiI6MTcyMTExMzE0NCwicGF0aCI6Ii8yOTcxNTg2NS8zMDAxODY3NDctMDFkOGY3ZGItZDY3Ny00YjZiLWJkMzktYThiOGZhMjE5NDQ2LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzE2VDA2NTkwNFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWU5YTBlNjQyNGY4MTljMDQ5Mjc3MWNkYjhiYzQwZjI4NTgyMTg3NjczNzQ4YjQ1ZmM4M2EzODYzMDcxNTIwNTEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.kLJ-SS-dObJSZhaCv8giYzCfP4u_6e2j3iSW-Xy0WtQ)