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.

Sign up for GitHub

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

Jump to bottom

SPPを導入すると中心重力問題が解けていない #53

Open
LWisteria opened this issue Mar 17, 2020 · 3 comments
Open

SPPを導入すると中心重力問題が解けていない #53

LWisteria opened this issue Mar 17, 2020 · 3 comments
Assignees
@LWisteria
Labels
bug Something isn't working

Comments

@LWisteria
Copy link
Contributor

#41 の作業中に気づいたのですが、現状のSPPが導入されたmasterで中心重力問題を実行すると、水面の動揺が収まりません。

GC法の時と同じようになっています。
@LWisteria LWisteria added the bug Something isn't working label Mar 17, 2020
@LWisteria LWisteria self-assigned this Mar 17, 2020
@LWisteria
Copy link
Contributor Author

よく考えたら #27 で実装された時には中心重力問題を解いていなかったようなので、確認漏れですかね

@LWisteria
Copy link
Contributor Author

なんでこうなっているか考えられるのは

  • SPP法の実装バグ
  • 粒子数密度が足りないところに強制的に足していることになるので、内部で勝手に運動量が生成されると言われるとそうかも（ほんとか？

とりあえずSPP法のバグの可能性がないか単体試験を追加してみるのが良さそう

@LWisteria
Copy link
Contributor Author

結果を貼っておきます（時間短縮のため、半径を100から20にして粒子数を減らしています）。

  • SPPあり（現master 82d0462 そのまま） _ 0500
  • SPPなし _ 0500

静止画にすると問題なさげに見えるかもですが、画像リンクから飛べる動画で見ると分かる通り、SPPありは振動しています

This was referenced Mar 17, 2020
Draft
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LWisteria LWisteria

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@LWisteria