-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
PR: add pictorialbar chart type and fix some bugs #1158
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #1158 +/- ##
==========================================
+ Coverage 86.57% 86.94% +0.36%
==========================================
Files 90 92 +2
Lines 1981 2014 +33
==========================================
+ Hits 1715 1751 +36
+ Misses 266 263 -3
Continue to review full report at Codecov.
|
pyecharts/charts/basic_charts/geo.py
Outdated
@@ -53,8 +53,9 @@ def add( | |||
self._zlevel += 1 | |||
|
|||
data = [] | |||
if type_ == ChartType.LINES and self._coordinate_system == "bmap": | |||
data = data_pair | |||
if self._coordinate_system == "bmap": |
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.
GeoChartBase shall not take care of the details of child. Otherwise, all children come back to base for help.
else: | ||
lng, lat = self.get_coordinate(n) | ||
data.append({"name": n, "value": [lng, lat, v]}) | ||
data = self._feed_data(data_pair, type_) |
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.
这样如何?
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.
嗯 oop 的性能用上了。
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.
Good to go
本次 PR 内容,