Skip to content

Fall cleanup#140

Merged
malihass merged 19 commits intomainfrom
cleanup
Oct 20, 2025
Merged

Fall cleanup#140
malihass merged 19 commits intomainfrom
cleanup

Conversation

@malihass
Copy link
Collaborator

@malihass malihass commented Oct 17, 2025

  • Use pathlib instead of makedirs to handle intermediate directory creation
  • Remove the folderManagement module
  • Remove label plot module (was only used for sparger paper)
  • Remove older sys.path.append
  • Fix warning logs
  • Make sure size group are correctly read
  • Add test for size group reading
  • Make bissection tool more general
  • Fix CI (move from miniconda to mamba)

@malihass malihass marked this pull request as draft October 17, 2025 15:15
@malihass malihass marked this pull request as ready for review October 19, 2025 19:14
@malihass malihass requested review from hsitaram and mjrahimi October 19, 2025 19:14
@malihass malihass merged commit 64f7a46 into main Oct 20, 2025
11 checks passed
@malihass malihass deleted the cleanup branch October 20, 2025 17:26
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants