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.

Sign up for GitHub

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

Jump to bottom

Fix PR #197 #198

Merged
merged 1 commit into from
Nov 15, 2023
Merged

Fix PR #197 #198

merged 1 commit into from
Nov 15, 2023

Conversation

andthum
Copy link
Owner

@andthum andthum commented Nov 15, 2023

Fix PR #197

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

Fix the search algorithms for finding the first column(s) and first row(s) of a hexagonal lattice.

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.

@andthum
[axial_hex_distribution_*nn.py]: Bugfix
7977f33 
Fix the search algorithms for finding the first column(s) and first
row(s) of a hexagonal lattice.
@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
@andthum andthum merged commit de3b404 into main Nov 15, 2023
19 checks passed
@andthum andthum deleted the fix/scripts/structure/axial-hex-distribution branch November 15, 2023 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bugfix Pull requests that fix a bug patch Change that results in a new patch version
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@andthum