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

[country map] Country Map updates from #3946 broke USA map #4112

Closed
3 tasks done
mjdepalma opened this issue Dec 23, 2017 · 3 comments · Fixed by #4121
Closed
3 tasks done

[country map] Country Map updates from #3946 broke USA map #4112

mjdepalma opened this issue Dec 23, 2017 · 3 comments · Fixed by #4121

Comments

@mjdepalma
Copy link

Make sure these boxes are checked before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if any
  • I have reproduced the issue with at least the latest released version of superset
  • I have checked the issue tracker for the same issue and I haven't found one similar

Superset version

0.22 and latest build

Expected results

USA map shows data

Actual results

no data shown

Steps to reproduce

  1. git checkout 0.22 or latest on master, build and run
  2. Upload attached .csv data
  3. Generate country_map chart using user_region for query parameter, and sum_pageviews as metric
  4. Observe no data shown on map
  5. git revert 16ab696
  6. build and run
  7. Observe that map data is shown on map

country_map_usa_sample_data.csv.gz

@xrmx
Copy link
Contributor

xrmx commented Dec 23, 2017

Maybe unrelated but US-VLI does not match https://en.wikipedia.org/wiki/ISO_3166-2:US

@xrmx
Copy link
Contributor

xrmx commented Dec 23, 2017

Any warning / error in the browser console log?

@mjdepalma
Copy link
Author

No exceptions are seen in the browser console. I also find that if I view the data in bar chart form (which works fine with the data), the same output is seen in the console as with the country map.

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 a pull request may close this issue.

2 participants