code clean up

[orbeckst]

Fixes #

Changes made in this Pull Request:

• use direct assignment instead of setattr
• moved util.run_residue to gibbs.run_residue

PR Checklist

• Tests?
• Docs?
• CHANGELOG updated?
• Issue raised/referenced?

[codecov]

Codecov Report

❌ Patch coverage is 23.07692% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 20.34%. Comparing base (1072f38) to head (a4df1e7).
⚠️ Report is 5 commits behind head on main.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull Request Overview

This PR refactors code to improve readability and organization by replacing setattr calls with direct attribute assignment and relocating a utility function to its appropriate module.

• Replaced setattr calls with direct assignment (obj.attr = value) for better readability
• Moved run_residue function from util.py to gibbs.py where it is more appropriately placed
• Added proper docstring and import statement for the relocated function

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
basicrta/util.py	Removed the run_residue function
basicrta/gibbs.py	Added run_residue function with documentation and replaced setattr with direct assignment
basicrta/cluster.py	Replaced setattr calls with direct attribute assignment

Comments suppressed due to low confidence (1)

basicrta/gibbs.py:1

• This import statement should be removed since run_residue has been moved to this module and is no longer available in util.py.

import os

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}