chore: using 'Py' instead of 'py' to be consistent

[themaherkhalil]

No description provided.

[github-actions]

@CodiumAI-Agent /describe

[QodoAI-Agent]

Title

chore: using 'Py' instead of 'py' to be consistent

---

PR Type

Other

---

Description

Update folder case to 'Py'
Use 'Py' in insight path construction
Fix fallback base folder path

---

Changes walkthrough 📝

	Relevant files
Configuration changes	PyTranslator.java Consistent `Py/Temp/` folder naming                                            src/prerna/ds/py/PyTranslator.java Changed folder path from /py/Temp/ to /Py/Temp/ Updated both insight-based and fallback paths +1/-1

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[github-actions]

@CodiumAI-Agent /review

[QodoAI-Agent]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Case Sensitivity Changing the folder path from "/py/Temp/" to "/Py/Temp/" may break deployments on case-sensitive file systems if the actual directory name does not match. Verify the directory exists with the updated casing. pyTemp = this.insight.getInsightFolder().replace('\\', '/') + "/Py/Temp/"; Consistent Path Updates Ensure all other references to the "py" folder in the codebase are updated to "Py" to prevent mismatches and runtime failures. pyTemp = this.insight.getInsightFolder().replace('\\', '/') + "/Py/Temp/";

[github-actions]

@CodiumAI-Agent /improve

[QodoAI-Agent]

PR Code Suggestions ✨

Category	Suggestion	Impact
Possible issue	Ensure temp directory exists Ensure the target directory exists before use by creating it if necessary to prevent runtime errors when writing files into it. src/prerna/ds/py/PyTranslator.java [174] pyTemp = (Utility.getBaseFolder() + "/Py/Temp/").replace('\\', '/'); +new File(pyTemp).mkdirs(); Suggestion importance[1-10]: 7 __ Why: Adding new File(pyTemp).mkdirs() prevents potential runtime errors by guaranteeing the pyTemp directory exists before writing files.	Medium
General	Use Path API for portable paths Use Java’s Path API instead of manual string concatenation to build the path, which avoids platform‐specific separator issues and improves readability. src/prerna/ds/py/PyTranslator.java [172] -pyTemp = this.insight.getInsightFolder().replace('\\', '/') + "/Py/Temp/"; +Path tempDir = Paths.get(this.insight.getInsightFolder(), "Py", "Temp"); +pyTemp = tempDir.toString().replace('\\', '/'); Suggestion importance[1-10]: 5 __ Why: Switching to Java’s Path API improves path construction and readability, but it’s a stylistic enhancement and is only applied in one branch.	Low
	Normalize path separators consistently Move the .replace('\', '/') call to after the full concatenation so that both the base folder and the appended segments are normalized uniformly. src/prerna/ds/py/PyTranslator.java [172] -pyTemp = this.insight.getInsightFolder().replace('\\', '/') + "/Py/Temp/"; +pyTemp = (this.insight.getInsightFolder() + "/Py/Temp/").replace('\\', '/'); Suggestion importance[1-10]: 3 __ Why: Moving .replace('\\', '/') after the full concatenation has minimal functional impact since the appended segments already use forward slashes.	Low

[github-actions]

@CodiumAI-Agent /update_changelog