Skip to content

Move forcing script from fluxnet experiments to src #1238

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

kmdeck
Copy link
Member

@kmdeck kmdeck commented Jul 17, 2025

Purpose

This moves the content of met_drivers_fluxnet and data_tools into src, with some cleanup and alteration of those functions.

Some notable changes:

  • we use the interpolating ability of TVI to handle gaps in data, rather than filling missing values by hand, except for in the plotting, which we can revisit in another PR.
  • remove metdrivers fluxnet script in favor of prescribed_forcing_fluxnet function in a new extension
  • set IC is a new function in the extension
  • get comparison data is in the new extension, this is used in plotting.

To-do

Future PR: Move plotting to LandSimulationVisualization, clean that up

Content


  • I have read and checked the items on the review checklist.

@kmdeck kmdeck force-pushed the kd/fluxnet_forcing branch from 69af2b9 to 587ffe8 Compare July 22, 2025 18:12
@juliasloan25 juliasloan25 mentioned this pull request Jul 24, 2025
3 tasks
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.

3 participants