Skip to content

Conversation

@ax3l
Copy link
Member

@ax3l ax3l commented Jul 29, 2015

I overlooked a few types in #899 (dev only) which should cause the position_SI argument always to
be zero and the cellSize_SI argument to potentially overflow (or at least to loose precision).

This pull request fixes that.

@ax3l ax3l added bug a bug in the project's code component: core in PIConGPU (core application) labels Jul 29, 2015
@ax3l ax3l added this to the Open Beta milestone Jul 29, 2015
@ax3l ax3l force-pushed the fix-gasFreeFormulaTypes branch from 91d504a to 474de77 Compare July 29, 2015 17:02
@PrometheusPi PrometheusPi self-assigned this Jul 30, 2015
@PrometheusPi
Copy link
Member

@ax3l: please add your name to the license header of freeFormulaImpl.hpp

@ax3l ax3l force-pushed the fix-gasFreeFormulaTypes branch 2 times, most recently from ce568d2 to deef745 Compare July 30, 2015 11:53
I overlooked a few types in ComputationalRadiationPhysics#899 (`dev` only)
which should cause the `position_SI` argument always to
be zero and the `cellSize_SI` argument to potentially
overflow (or at least to loose precision).

This commit fixes that.
@ax3l ax3l force-pushed the fix-gasFreeFormulaTypes branch from 1fc8f43 to de6f158 Compare July 30, 2015 12:07
@ax3l
Copy link
Member Author

ax3l commented Jul 30, 2015

updated and pushed :)

@n01r n01r mentioned this pull request Jul 30, 2015
1 task
@ax3l
Copy link
Member Author

ax3l commented Jul 30, 2015

@PrometheusPi merg-able :)

PrometheusPi added a commit that referenced this pull request Jul 30, 2015
Fix Gas Profile: Free Formula Types
@PrometheusPi PrometheusPi merged commit 9e45665 into ComputationalRadiationPhysics:dev Jul 30, 2015
@ax3l ax3l deleted the fix-gasFreeFormulaTypes branch July 30, 2015 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug a bug in the project's code component: core in PIConGPU (core application)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants