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

Clean up main.py, in preparation for integrating user-facing features #67

Merged
merged 7 commits into from
Jun 29, 2024

Conversation

rbyrne299
Copy link
Contributor

@rbyrne299 rbyrne299 commented Jun 27, 2024

  • Still holding off on integration tests in test_main that actually check output; waiting until this output is stable/finalized enough
  • Failing Fibonacci tests have been moved to separate fix-fib branch; fixes will be made in future PR as Ising is the priority

…sition

- Removed anyon and model commands from help message
- Removed shell command, as doesn't seem necessary
- Updated anyon init loop parsing to allow for space in 2D position
- Handle error from 2D position with >2 coordinates
- Test for unhandled error given invalid position inputs
@rbyrne299 rbyrne299 requested a review from Haadi-Khan June 27, 2024 22:08
@rbyrne299 rbyrne299 requested a review from juli6nne June 29, 2024 19:27
@rbyrne299 rbyrne299 merged commit b3ec6a4 into main Jun 29, 2024
1 check passed
@rbyrne299 rbyrne299 deleted the clean-up-main.py branch June 29, 2024 22:04
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.

2 participants