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

Introducing DensityMapbox #163

Merged
merged 6 commits into from
May 10, 2024
Merged

Introducing DensityMapbox #163

merged 6 commits into from
May 10, 2024

Conversation

gwbres
Copy link
Contributor

@gwbres gwbres commented Sep 21, 2023

Density Mapboxes are very important graph objects to display datasets over an entire or a portion of the world map

@gwbres
Copy link
Contributor Author

gwbres commented Sep 21, 2023

I use this in one of my application, to draw the evolution of the TEC in Ionosphere, throughout one day

image

image

@gwbres gwbres force-pushed the density-mapbox branch 2 times, most recently from dcbee20 to 1d0e231 Compare September 22, 2023 13:09
Signed-off-by: Guillaume W. Bres <guillaume.bressaix@gmail.com>
@gwbres gwbres marked this pull request as ready for review September 22, 2023 13:38
@gwbres
Copy link
Contributor Author

gwbres commented Oct 17, 2023

Hello @igiagkiozis,

any chances to get a review on this ?

@gwbres
Copy link
Contributor Author

gwbres commented Nov 28, 2023

@igiagkiozis, @mfreeborn

any chances to have this at least discussed ?

@gwbres
Copy link
Contributor Author

gwbres commented Dec 10, 2023

up @igiagkiozis, @mfreeborn

@gwbres
Copy link
Contributor Author

gwbres commented Jan 9, 2024

will most likely (when I don't have better things to do) publish a secondary library of this, since this is not maintained

@gwbres
Copy link
Contributor Author

gwbres commented May 2, 2024

Hello,

@andrei-ng any chances to discuss this in a near future ? no emergency though.
One of my app would like to see the density plots unlocked at some point

@andrei-ng
Copy link
Collaborator

andrei-ng commented May 2, 2024

Hello,

@andrei-ng any chances to discuss this in a near future ? no emergency though. One of my app would like to see the density plots unlocked at some point

@gwbres Yes, I will have a look at it in the second part of next week. Let's discuss then. Can you in the meantime rebase it on main so the CI is triggered.

@andrei-ng andrei-ng self-requested a review May 10, 2024 09:48
@andrei-ng andrei-ng self-assigned this May 10, 2024
Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>
Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>
Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>
Copy link
Collaborator

@andrei-ng andrei-ng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gwbres looks good to me.

 - apply target configuration to StaticPlotTemplate

Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com>
@andrei-ng andrei-ng merged commit 852ed2e into plotly:main May 10, 2024
17 of 18 checks passed
@andrei-ng andrei-ng mentioned this pull request May 10, 2024
@gwbres
Copy link
Contributor Author

gwbres commented May 10, 2024

@andrei-ng

thank you very much for looking into this, sorry i'm currently away and was unable to rebase this branch

see you

@andrei-ng
Copy link
Collaborator

@andrei-ng

thank you very much for looking into this, sorry i'm currently away and was unable to rebase this branch

see you

No worries. Happy I was able to update your branch

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.

2 participants