Skip to content
Snippets Groups Projects

example IO refactoring ideas

Closed gcorradini requested to merge conus-dps-regional-refactor into conus-dps

I went through the exercise of reviewing the diffs between conus-dp and main to understand what MAAPY (you like that :smirk:?) inclusions looked like. Here's a quick summary:

  1. Most of our changes are due to some linter that Eli must be using. That's good and bad. Good b/c we should use linters! Bad because it makes the changes seem like a lot more

  2. FileIO.py has the most changes just like Eli expected. A lot of the changes add some regional pass-through args. This MR proposes some ways to handle that but be backward compatible so we don't have to change the functionality of the original code at all. If we use some type of strong config/settings file (like jupyterhub codebase and others like it do) we can have a powerful way to be able to run this code in "different contexts/runtimes/environments" without changing the main transform/algorithms. :fire:

  3. Didn't do an intensive sweep so there might be more patterns we need to apply or get acrobatic about

Edited by gcorradini

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading