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

Refactor receiver-variables #85

Open
MarleneBusch opened this issue Sep 20, 2024 · 1 comment
Open

Refactor receiver-variables #85

MarleneBusch opened this issue Sep 20, 2024 · 1 comment
Assignees
@kalebphipps @MarleneBusch
Labels
maintenance Improve code readability and robustness.
Milestone
ARTIST Release 1.0

Comments

@MarleneBusch
Copy link
Collaborator

Possibly misleading variables names
Right now everything in ARTIST that concerns the tower is called receiver.
This is misleading because some variables reference the receiver but some reference the calibration target.

Expected behavior
Differentiate between the receiver and the calibration target.

Additional context
Refactoring in PyCharm is easier than in VSCode .
@MarleneBusch MarleneBusch added the maintenance Improve code readability and robustness. label Sep 20, 2024
@kalebphipps
Copy link
Member

This Issue should be solved before ARTIST 1.0 release and is therefore now a part of the ARTIST 1.0 release milestone!

@kalebphipps kalebphipps added this to the ARTIST Release 1.0 milestone Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kalebphipps kalebphipps

@MarleneBusch MarleneBusch

Labels
maintenance Improve code readability and robustness.
Projects
None yet
Milestone
ARTIST Release 1.0
Development

No branches or pull requests
2 participants
@kalebphipps @MarleneBusch