[dennou-ruby:003734] Re: GPhys.join での associate coordinate

[Takeshi Horinouchi <horinout@xxxxxxxxxxxxxxxxx>]

西澤さま：

ありがとうございます。コミットして (本尊にpushして) おきました。

既存の問題解決ですから，ブランチを切る必要はないです。
なにか実験的に新しい機能を作る場合はブランチ作成が
お勧めですが，そのブランチを本尊にpushするかどうかは
任せます（開発コードを共有したいかどうかで決めるとか）。
git レポジトリのローカルルールは
http://www.gfd-dennou.org/arch/ruby/products/gphys
/develop/git_repo_howto.htm
ですが，ブランチの記述はあっさりすぎですかね？

> 堀之内様
> 
> GPhys.join した場合に、 associate coordinate は未対応ですが、
> 実装してみました。
> GPhys.join を使っているので効率的ではない気がしますが、
> とりあえず動作しているようです。
> 
> 問題無さそうであれば取り込んでいただけますでしょうか。
> よろしくお願いいたします。
> 
> # gphys のコミットポリシーをよく把握していませんが、
> # このようにメールでパッチをお送りするのがよいでしょうか?
> # それとも feature branch を切って、コミット（&メールで報告）するのがよろしいでしょうか？
> 
> 
> 西澤誠也
> 
> 
> 
> --- a/lib/numru/gphys/gphys.rb
> +++ b/lib/numru/gphys/gphys.rb
> @@ -1145,6 +1145,12 @@ module NumRu
>          grid = Grid.new(*axes)
>          # ** Develpment Note (2013-03-02, horinout)**  joining assoc_coords
>          # is yet to be supported; not difficult but cumbersome
> +        if gp0.assoc_coords
> +          assoc_coord = gp0.assoccoordnames.collect do |aname|
> +            GPhys.join( gpnary.collect{|gp| gp.assoc_coord_gphys(aname)} )
> +          end
> +          grid.set_assoc_coords(assoc_coord)
> +        end
> 
>          #< data >
>          data = VArrayComposite.new( gpnary.collect{|gp| gp.data} )
> 
> -- 
> Seiya Nishizawa
> RIKEN Advanced Institute for Computational Science
> Tel: +81-78-940-5754, Fax: +81-78-304-4972
> 7-1-26, Minatojima-minami-machi, Chuo-ku, Kobe, Hyogo 650-0047, Japan
> 

堀之内 武
北海道大学 地球環境科学研究院 地球圏科学部門
〒060-0810 札幌市北区北10条西5丁目