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

Fixed process of building grid #58

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Fixed process of building grid #58

wants to merge 5 commits into from

Conversation

kolayne
Copy link
Member

@kolayne kolayne commented Nov 24, 2021

Changed the algorithm of checking if a particle is located on a face (which seems to have fixed the grid creation issues) + minor meta fixes.

Please, confirm, but as far as I can see, this PR fixes #29, fixes #31, fixes #38.

Note that the checks with the Ubuntu 16.04 operating system will fail because GitHub does not any longer provide containers with it. This is already fixed in #55, so I can merge it into this branch as soon as you approve it.

tanya-kta and others added 5 commits November 20, 2021 20:04
…nstead of previous approach.

This might have fixed one or many of: #29, #31, #38

Co-authored-by: Nikolay Nechaev <Nikolay_Nechaev@mail.ru>
…t only squares

Co-authored-by: Tanya Kadykova <tanya-kta@bk.ru>
…ion similar

Co-authored-by: Tanya Kadykova <tanya-kta@bk.ru>
Co-authored-by: Tanya Kadykova <tanya-kta@bk.ru>
Co-authored-by: Tanya Kadykova <tanya-kta@bk.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Illegal memory access inside of SimulationMap constructor Bus error (SIGBUS) Stack smashing (SIGABRT)
2 participants