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
Geolines data value #566
Geolines data value #566
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #566 +/- ##
==========================================
+ Coverage 98.89% 98.89% +<.01%
==========================================
Files 96 96
Lines 3700 3710 +10
==========================================
+ Hits 3659 3669 +10
Misses 41 41
Continue to review full report at Codecov.
|
@@ -161,6 +170,7 @@ def add( | |||
"symbolSize": 10, | |||
"data": _data_scatter, | |||
"label": chart["label"], | |||
"tooltip": {"formatter": "{b}"}, |
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.
tooltip formatter could a function
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.
考虑过这点,这里只是用于 scatter 图中点的 tooltip,该点的数值无意义,所以只显示城市名即可。没必要提供参数让用户修改。
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.
OK
pyecharts/charts/geolines.py
Outdated
_from_name, _to_name = d | ||
for element in data: | ||
assert len(element) >= 2 | ||
_lines_value = None |
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.
_line_value please, it is a single value.
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.
done
test/test_geolines.py
Outdated
"从广州出发", data_guangzhou, tooltip_formatter="{a} : {c}", **style_geo | ||
) | ||
lines.render() | ||
html_content = lines._repr_html_() |
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.
use get_default_rendering_file_content()
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.
done
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.
GTG
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
* Update: 更新 x,y 轴 formatter 参数注释 * #565 Geolines 图数据项可以新增数值维度 * Update: charts docs update * Fix: fixed bronken test * Update: 更新 github ISSUE 及 PR 提交模板 * Update: lines -> line * Update: 删除无用语句
本次 PR 内容,