Closed AbdullahKazi500 closed 1 month ago
Hey, thanks for the PR. This is a great start, but there are at least the following few things to fix:
I am marking this PR a draft. Feel free to switch it back, once you think it's are ready for a review. (I also unchecked the two checkboxes in the PR message that says tests are passing. Feel free to check them again, once you completed the test, and indeed they are passing.)
Thanks @peterkomar-aws sure I will keep working on this
Please consolidate the work to this PR: https://github.com/amazon-braket/amazon-braket-sdk-python/pull/989
Issue
fixes #969
Description of changes:
Added factory methods to the AtomArrangement class to create various lattice arrangements, including square, rectangular, honeycomb, Bravais, and decorated Bravais lattices.
Testing done:
Merge Checklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.General
Tests
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.