Skip to content

feat: refactor transfer_all function and add transferable_wells utility#455

Merged
jirhiker merged 1 commit intostagingfrom
BDMS-520-1-1-Cleanup-2.0
Jan 31, 2026
Merged

feat: refactor transfer_all function and add transferable_wells utility#455
jirhiker merged 1 commit intostagingfrom
BDMS-520-1-1-Cleanup-2.0

Conversation

@jirhiker
Copy link
Member

Why

This PR addresses the following problem / context:

  • Use bullet points here

How

Implementation summary - the following was changed / added / removed:

  • Use bullet points here

Notes

Any special considerations, workarounds, or follow-up work to note?

  • Use bullet points here

Copilot AI review requested due to automatic review settings January 31, 2026 13:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refactors the transfer_all function to simplify its interface and moves configuration management inside the function. It also introduces a new utility script for identifying transferable wells.

Changes:

  • Simplified transfer_all function signature by removing parameters and managing configuration internally via environment variables
  • Removed unused profile_waterlevels parameter from _run_continuous_water_level_transfers
  • Added transferable_wells.py utility script to extract and export transferable wells data

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
transfers/transferable_wells.py New utility script that reads well data, filters transferable wells, and exports to CSV
transfers/transfer.py Refactored transfer_all to simplify interface and manage configuration internally

@jirhiker jirhiker merged commit 762d535 into staging Jan 31, 2026
6 checks passed
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.

1 participant