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

Improve map rendering #3

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Improve map rendering #3

wants to merge 12 commits into from

Conversation

godsic
Copy link
Contributor

@godsic godsic commented Jul 14, 2022

by making Cesium fetch higher resolution tiles more aggressively as
suggested in CesiumGS/cesium#3279 (comment)

image

@ckoval7
Copy link
Owner

ckoval7 commented Jul 15, 2022

A value of 1.0 causes a noticeable hit in performance. Perhaps the recommended value of 1.33 would be better as recommended in the issue you linked, as well as a related issue.

@godsic
Copy link
Contributor Author

godsic commented Jul 16, 2022

@ckoval7 Aha, I have not noticed any regressions on my average Intel NUC, but perhaps lower power hardware might straggle a bit indeed. Perhaps a better idea is to add checkbox that toggles higher quality map rendering as I did in my latest commit?

@godsic
Copy link
Contributor Author

godsic commented Aug 3, 2022

@ckoval7 I might forgot to mention, but when I force pushed I added the button I've mentioned. My bet, should not force push ever) Anyway, that's how it looks like:
image

@ckoval7
Copy link
Owner

ckoval7 commented Aug 5, 2022

I am going to implement this a little differently. It'll be entirely in the browser, not having to touch the python side of things.

@godsic
Copy link
Contributor Author

godsic commented Aug 6, 2022

Sure, I am not an expert in web development at all, just mimicked how other buttons were implemented) does not make sense to have a state for such thing on the server side.

@alphafox02
Copy link

With a little extra leg work, it's possible to run offline maps w/ DFA. I just need a little help to make the process easier. Here's the current process, just replace PhotonMap w/ DFA.

https://www.youtube.com/watch?v=dZu03Wcisv4

by making Cesium fetch higher resolution tiles more aggressively as
suggested in
CesiumGS/cesium#3279 (comment)
Instead keep the last know measurement and plot corresponding lob in
gray color to alert operator. This change is suggested to be used
together with krakenrf/krakensdr_doa#72 .
@IIvanIvanovich
Copy link

Hi! Will these changes be included in the main branch? I want to try them at work.

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 this pull request may close these issues.

None yet

5 participants