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

Fix axial_hex_distribution_*nn.py #197

Merged
merged 5 commits into from
Nov 15, 2023

Conversation

andthum
Copy link
Owner

@andthum andthum commented Nov 15, 2023

Fix axial_hex_distribution_*nn.py

Type of change

  • Change of core package.
  • Change of scripts.
  • Bug fix.
  • New feature.
  • Code refactoring.
  • Dependency update.
  • Documentation update.
  • Maintenance.
  • Other: Description.
  • Non-breaking (backward-compatible) change.
  • Breaking (non-backward-compatible) change.

Proposed changes

Scripts axial_hex_distribution_1nn.py and axial_hex_distribution_2nn.py: Fix the search algorithms for finding the first column(s) and row(s) of a hexagonal lattice when the lattice faces lie (almost) exactly on the edges of the simulation box.

PR checklist

  • I followed the guidelines in the Developer's guide.
  • New/changed code is properly tested.
  • [~] New/changed code is properly documented.
  • The CI workflow is passing.

@github-actions github-actions bot added bugfix Pull requests that fix a bug patch Change that results in a new patch version labels Nov 15, 2023
* Sort import statements according to isort.
* Format the code and docstrings of the defined functions.
Fix the search algorithms for finding the first column (function
`get_1st_hex_face_col`) and the first two staggered rows (function
`get_1st_hex_face_rows`) of a hexagonal lattice.
* Sort import statements according to isort.
* Format the code and docstrings of the defined functions.
Fix the search algorithms for finding the first two staggered columns of
a hexagonal lattice (function `get_1st_hex_face_cols`).
@andthum andthum force-pushed the fix/scripts/structure/axial-hex-distribution branch from 35a5ef5 to be89686 Compare November 15, 2023 20:49
@andthum andthum merged commit fead2c3 into main Nov 15, 2023
18 checks passed
@andthum andthum deleted the fix/scripts/structure/axial-hex-distribution branch November 15, 2023 21:04
@andthum andthum mentioned this pull request Nov 15, 2023
14 tasks
andthum added a commit that referenced this pull request Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Pull requests that fix a bug patch Change that results in a new patch version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant