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

Normalizing the coordinates #4

Closed
jklae opened this issue Aug 18, 2021 · 0 comments
Closed

Normalizing the coordinates #4

jklae opened this issue Aug 18, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@jklae
Copy link
Owner

jklae commented Aug 18, 2021

Description
Currently, the drawing coordinates and the simulation coordinates are the same. It is not good because the smaller the grid scale, the larger the calculation error. Also, some coordinates are negative to center the grid position. These coordinates are very cumbersome in simulation calculations, especially when interpolating particle variables. Therefore, the simulation calculations will be more efficient by having the drawing coordinates and the simulation coordinates separately.

Todo

  • Do not equate simulation coordinates with drawing coordinates.
  • Normalize the simulation coordinates [0, N+2]. These coordinates make simulation calculations convenient.
  • Remove variables like stride and offset. As long as the coordinates are normalized, these variables are not needed.
  • Modify the interpolation function. In the interpolation function, the offset and stride of the particle are calculated. Since the coordinate system has changed, these must also be corrected.

Note
A method for compensating for the difference between the two coordinate systems is presented in jklae/DXViewer#25.
This issue is derived from #3.

@jklae jklae added the enhancement New feature or request label Aug 18, 2021
@jklae jklae closed this as completed Aug 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant