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.

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

unrealistic bias in Z velocities. #6

Closed
Jwely opened this issue Feb 3, 2016 · 1 comment
Closed

unrealistic bias in Z velocities. #6

Jwely opened this issue Feb 3, 2016 · 1 comment
Labels

Comments

@Jwely
Copy link
Owner

Jwely commented Feb 3, 2016

Uncertainty analysis is showing an exceptionally strong bias in the Z velocity measurements, which gets better as the interrogation plane moves down stream.

This bias can be checked for by examining the mean axial velocity measured in each vortex compared to the free stream velocity. In every single case, the mean axial velocity is slightly elevated from the free stream velocity measured with dynamic pressure, precisely as we would expect. This seems to indicate that in reality, Z measurements are not significantly biased.

One attempted fix is to remove the meshgrid recalculation for a small z displacement in the ArtificialPIV instance which generates the image pairs. We will see how it goes, but must wait yet many more hours for it to finish computing.

Jwely added a commit that referenced this issue Feb 3, 2016
This is an attempted fix. issue #6
Jwely added a commit that referenced this issue Feb 4, 2016
Jwely added a commit that referenced this issue Feb 4, 2016
This reverts commit b6f4b99.

Commited in case i want to get these images back, but the attempted fix definitely did not help, it had the initially expected outcome of making things worse. Reverting.
Jwely added a commit that referenced this issue Feb 4, 2016
attempted issue #6 fix did not work.
@Jwely
Copy link
Owner Author

Jwely commented Feb 9, 2016

After extensive review, no flaws can be found in the classes responsible for the creation of artificial particle images. Perhaps the simulation criteria is a little bit too drastically worse case, but no more time can be dedicated to discovering the issue. The results of the uncertainty analysis have proven useful and consistent in all maters pertaining to the X and Y velocity fluctuations. This issue will not be fixed by the graduation release.

@Jwely Jwely added the wontfix label Feb 29, 2016
@Jwely Jwely closed this as completed Feb 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant