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

Minor logic errors on qt3scan #158

Closed
vasilisniaouris opened this issue Mar 23, 2024 · 0 comments · Fixed by #159
Closed

Minor logic errors on qt3scan #158

vasilisniaouris opened this issue Mar 23, 2024 · 0 comments · Fixed by #159
Assignees
Labels
code improvement GUI work this work involves GUI components invalid This doesn't seem right

Comments

@vasilisniaouris
Copy link
Collaborator

vasilisniaouris commented Mar 23, 2024

  1. The full range of x and y is not scanned. For both axes the last point is neglected.
  2. The scan-stop logic is wrong. The stop method of the controller is called twice, once to stop the scan and one to perform post-stop actions. These functionalities need to be separated in two different actions.
@vasilisniaouris vasilisniaouris added invalid This doesn't seem right GUI work this work involves GUI components labels Mar 23, 2024
@vasilisniaouris vasilisniaouris self-assigned this Mar 23, 2024
@vasilisniaouris vasilisniaouris changed the title Minor bugs on qt3scan Minor logic errors on qt3scan Mar 23, 2024
@vasilisniaouris vasilisniaouris linked a pull request Mar 23, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code improvement GUI work this work involves GUI components invalid This doesn't seem right
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant