Sweep: Refactor ableton.py to be cleaner, extracting any methods you can into separate files if that makes sense

[axeldelafosse]

Sweep: Refactor ableton.py to be cleaner, extracting any methods you can into separate files if that makes sense

[sweep-ai]

Here's the PR! #26.

💎 Sweep Pro: I used GPT-4 to create this ticket. You have 53 GPT-4 tickets left.

• Install Sweep Configs: Pull Request

---

Step 1: 🔍 Code Search

Working on it...

---

Step 2: 🧐 Snippet Analysis

From looking through the relevant snippets, I decided to make the following modifications:

File Path	Proposed Changes
ableton.py	Extract the functions say(text), switch_to_ableton(), and export(track, position) into a separate utility file. Encapsulate the global variables NAME, IS_RETINA, OS, PYTHON_EXEC, and STEMS into a class. Simplify the main() function by extracting some of its logic into separate functions.
ableton_utils.py	Create this file and add the functions say(text), switch_to_ableton(), and export(track, position) that were extracted from ableton.py. Also, add any other utility functions that are identified during the refactoring process.

---

Step 3: 📝 Planning

I have created a plan for writing the pull request. I am now working my plan and coding the required changes to address this issue. Here is the planned pull request:

Refactor ableton.py to be cleaner and extract methods into separate files
sweep/refactor-ableton-py

Description

This PR refactors the ableton.py file to improve code organization and readability. Several functions have been extracted into a separate utility file, ableton_utils.py, to make the code cleaner and more maintainable. The global variables in ableton.py have been encapsulated into a class for better organization. The main() function has also been simplified by extracting some of its logic into separate functions.

Summary of Changes

• Extracted the functions say(text), switch_to_ableton(), and export(track, position) from ableton.py into ableton_utils.py.
• Encapsulated the global variables NAME, IS_RETINA, OS, PYTHON_EXEC, and STEMS into a class in ableton.py.
• Simplified the main() function in ableton.py by extracting some of its logic into separate functions.
• Created the ableton_utils.py file to contain the extracted utility functions.
• Improved code organization and readability in ableton.py.

---

Step 4: ⌨️ Coding

I have finished coding the issue. I am now reviewing it for completeness.

---

Step 5: 🔁 Code Review

Success! 🚀

---

I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind!
^{Join Our Discord}