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

Chart Map: SccaterGeo UI #143

Closed
reza1615 opened this issue Apr 2, 2020 · 15 comments
Closed

Chart Map: SccaterGeo UI #143

reza1615 opened this issue Apr 2, 2020 · 15 comments

Comments

@reza1615
Copy link

reza1615 commented Apr 2, 2020

  • Projection dropdown window's width needs to be wider
    image

  • I extracted projections' svg. when user select one of them, in a picture box the image can be shown to help the user for selection the proper one.
    Projections.zip
    image

  • For SccaterGeo: It should accept empty Value (Just show some pins on the locations without any value)

@reza1615 reza1615 changed the title Chart Map UI Chart Map: SccaterGeo UI Apr 2, 2020
@aschonfeld
Copy link
Collaborator

Good suggestions

@aschonfeld
Copy link
Collaborator

I tried adding the projection samples to the app but the files were huge. So the best solution I could come up with was hosting this markdown file with links to all the svgs: https://github.com/aschonfeld/dtale-media/blob/master/projections_listing.md

Then I included an info link in the Projections label:
image

I couldn't have links directly to the svg files themselves because of the cross-scripting problems and I tried loading the files from the back-end but that fell over because of proxy problems and I really didn't want to support proxy settings for something so small.

@reza1615
Copy link
Author

reza1615 commented Apr 3, 2020

I made them smaller (height, size) and change it to png (Image extension) . please test them
Projections2.zip

@aschonfeld
Copy link
Collaborator

That is much better (.348 mb) but it would still make up about 5% of the egg if I include them as static files. If you think these thumbnails would be a big win I'll add them, but I would think the simple markdown I setup on github should be more than enough of a sample for users.

@reza1615
Copy link
Author

reza1615 commented Apr 3, 2020

1- In my opinion make the code more visual is more attractive
2-As you mention before we should reduce using internet
3-For each case going to help is annoying.

@aschonfeld
Copy link
Collaborator

One last piece to make note of is that the demo site I'm running right now on PythonAnywhere only has 512MB of space so every file we add to this makes a big difference. I'm trying to see if I can get someone to flip the bill on getting more space but as of right now those are the constraints.

@reza1615
Copy link
Author

reza1615 commented Apr 3, 2020

I made them gray-scale and reduce their size now total size is 136 K
Projections3.zip

@aschonfeld
Copy link
Collaborator

Ok, I'll integrate this and I'm close to getting my employers to get us some more space on PythonAnywhere :)

@aschonfeld
Copy link
Collaborator

One last issue, that extra space to the right of 'Projection' no longer exists since I've added grouping to maps as well. So the best I'll be able to do it is to continue with that '?' icon in the label that you can hover over once you've selected a value to view its thumbnail.

The best scenario would have been to embed the images into the dropdown options, but it looks like dash support that even though the component they are using "react-select" does actually support it.

@aschonfeld
Copy link
Collaborator

aschonfeld commented Apr 3, 2020

Sorry I know this isn't what you envisioned but I've spent way too much time on this one thing so it will have to do for now.

natural earth selcted:
image

mercator selected:
image

No selection:
image

@reza1615
Copy link
Author

reza1615 commented Apr 3, 2020

Thank you for your time that is Ok. for later here https://dash-gallery.plotly.host/dash-brain-viewer/ the color selector is also interesting.
image

@aschonfeld
Copy link
Collaborator

This would replace the "Colorscale" dropdown?

@reza1615
Copy link
Author

reza1615 commented Apr 3, 2020

Yes It can be for there
Also for these images we can have gallery of the images on a popup and simply user select one of them (Replace Colorscales with the images)

@reza1615
Copy link
Author

reza1615 commented Apr 3, 2020

https://github.com/plotly/dash-colorscales/blob/master/README.md

aschonfeld pushed a commit that referenced this issue Apr 4, 2020
* [#143](#143), scattergeo map chart UI changes
* updated offline chart generation of maps to work without loading topojson from the web
* fix to allow correlations timeseries to handle when date columns jump between rolling & non-rolling
* added slider to animation and added animation to maps
* fixes for IE 11 compatibility issues
* labeling changes for "Reshape" popup
* added grouping to maps
@aschonfeld aschonfeld mentioned this issue Apr 4, 2020
aschonfeld pushed a commit that referenced this issue Apr 4, 2020
* #143, scattergeo map chart UI changes
* updated offline chart generation of maps to work without loading topojson from the web
* fix to allow correlations timeseries to handle when date columns jump between rolling & non-rolling
* added slider to animation and added animation to maps
* fixes for IE 11 compatibility issues
* labeling changes for "Reshape" popup
* added grouping to maps
aschonfeld pushed a commit that referenced this issue Apr 4, 2020
* #143, scattergeo map chart UI changes
* updated offline chart generation of maps to work without loading topojson from the web
* fix to allow correlations timeseries to handle when date columns jump between rolling & non-rolling
* added slider to animation and added animation to maps
* fixes for IE 11 compatibility issues
* labeling changes for "Reshape" popup
* added grouping to maps
@aschonfeld
Copy link
Collaborator

added v1.8.3

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

No branches or pull requests

2 participants