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

"minY:0 だと0が反映されない。数値タイプのconfig値処理を変更。 #26

Closed
toshirot opened this Issue Aug 16, 2016 · 4 comments

Comments

Projects
None yet
1 participant
@toshirot
Owner

toshirot commented Aug 16, 2016

データの最小値が優先されてる
バグサンプル
http://ccchart.org/test/minY/test-v1.12.06.htm

thanx H.Iさん

これは Number型の config値 の時にたとえば
this.hoge = op.config.hoge || 10; などとしていると
op.config.hoge === 0 だった時に 0に決まらずに10になってしまうという理由です。

実は以前からこれを直そうと思っていたのですが、これを機会に メソッドを作って対応したいと思います
作る関数
ccchart.util.setConfigNum: function(it, propName, configVal, gfcVal, defaultVal)

util:{

  setConfigNum: function(it, propName, configVal, gfcVal, defaultVal){
    //暫定
    //数値タイプのコンフィグ値をセットする configVal||gfcVal では0の時にうまく動作しないので
    //defaultValが無ければundefinedを返す
    //e.g. setConfigNum(this, 'minY', this.op.config.minY, this.gcf.minY, 0)
    it[propName+'Default'] = defaultVal;
    if(typeof configVal === 'number') return it[propName] = configVal;
    if(typeof gfcVal === 'number') return it[propName] = gfcVal;
    if(typeof defaultVal === 'number')return it[propName] = defaultVal;
    return undefined;
  }

使用例(内部) 
//最小値 デフォルトundefined
this.minY = this.util.setConfigNum(this, 'minY', this.op.config.minY, this.gcf.minY)
//線幅 デフォルト2
this.lineWidth =
this.util.setConfigNum(this, 'lineWidth', this.op.config.lineWidth, this.gcf.lineWidth, 2);

@toshirot toshirot added the bug label Aug 16, 2016

@toshirot

This comment has been minimized.

Show comment
Hide comment
Owner

toshirot commented Aug 16, 2016

@toshirot

This comment has been minimized.

Show comment
Hide comment
@toshirot

toshirot Aug 17, 2016

Owner

今回 this.util.setConfigNumによる処理に変更するconfigプロパティ

width
height
markerWidth
maxWsColLen
minY
maxY
minX
maxX
innerCircle
outerCircle
lineWidth
pieHoleRadius
pieRingWidth
barWidth
borderWidth
xScaleXOffset
xScaleYOffset
xScaleRotate
colNamesTitleOffset
yScaleXOffset
yScaleYOffset
yScaleRotate
hanreiXOffset
hanreiYOffset
hanreisYSpace
hanreiRadius
defaultZindex

Owner

toshirot commented Aug 17, 2016

今回 this.util.setConfigNumによる処理に変更するconfigプロパティ

width
height
markerWidth
maxWsColLen
minY
maxY
minX
maxX
innerCircle
outerCircle
lineWidth
pieHoleRadius
pieRingWidth
barWidth
borderWidth
xScaleXOffset
xScaleYOffset
xScaleRotate
colNamesTitleOffset
yScaleXOffset
yScaleYOffset
yScaleRotate
hanreiXOffset
hanreiYOffset
hanreisYSpace
hanreiRadius
defaultZindex

@toshirot

This comment has been minimized.

Show comment
Hide comment
@toshirot

toshirot Aug 17, 2016

Owner

テスト開始
バグの出ているもの

http://ccchart.org/#102 //XY軸ともにNaN "type": "heatmap", > fixed at ccchart-v1.12.07b2.js
http://ccchart.org/#91 //X軸にNaN "type": "scatter", > fixed at ccchart-v1.12.07b2.js
http://ccchart.org/#79 //サイズが変わった "type": "pie", > fixed at ccchart-v1.12.07b3.js
http://ccchart.org/#53 //Y軸にNaN "type": "stackedarea", > fixed at ccchart-v1.12.07b3.js
http://ccchart.org/#4 //Y軸にNaN "type": "stacked", > fixed at ccchart-v1.12.07b2.js
http://ccchart.org/#93 //Y軸にNaN "type": "stacked", > fixed at ccchart-v1.12.07b2.js
etc...

やっぱ一度にいっぱいいじったからいろいろ変になってる。。。

Owner

toshirot commented Aug 17, 2016

テスト開始
バグの出ているもの

http://ccchart.org/#102 //XY軸ともにNaN "type": "heatmap", > fixed at ccchart-v1.12.07b2.js
http://ccchart.org/#91 //X軸にNaN "type": "scatter", > fixed at ccchart-v1.12.07b2.js
http://ccchart.org/#79 //サイズが変わった "type": "pie", > fixed at ccchart-v1.12.07b3.js
http://ccchart.org/#53 //Y軸にNaN "type": "stackedarea", > fixed at ccchart-v1.12.07b3.js
http://ccchart.org/#4 //Y軸にNaN "type": "stacked", > fixed at ccchart-v1.12.07b2.js
http://ccchart.org/#93 //Y軸にNaN "type": "stacked", > fixed at ccchart-v1.12.07b2.js
etc...

やっぱ一度にいっぱいいじったからいろいろ変になってる。。。

@toshirot toshirot changed the title from "minY:0 だと0が反映されない to "minY:0 だと0が反映されない。数値タイプのconfig値処理を変更。 Aug 20, 2016

@toshirot toshirot closed this Aug 21, 2016

@toshirot

This comment has been minimized.

Show comment
Hide comment
@toshirot

toshirot Aug 25, 2016

Owner

scatterにバグ thanx mosyさん

mosy> markerWidthのほうは1つ前のバージョンなら大丈夫ですが、useMarkerのほうは同じみたいです (2016/8/25 13:56:32)
mosy> ccchartの最新(1.12.07)でscatterを試しているのですが、markerWidthを指定しないとプロットされない、useMakerでcss-maruとするとデータが複数あっても全て同じ色になります (2016/8/25 13:55:14)

Owner

toshirot commented Aug 25, 2016

scatterにバグ thanx mosyさん

mosy> markerWidthのほうは1つ前のバージョンなら大丈夫ですが、useMarkerのほうは同じみたいです (2016/8/25 13:56:32)
mosy> ccchartの最新(1.12.07)でscatterを試しているのですが、markerWidthを指定しないとプロットされない、useMakerでcss-maruとするとデータが複数あっても全て同じ色になります (2016/8/25 13:55:14)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment