Compute land-atmosphere terrestrial coupling index in key_metrics #145
Task list completed / task-list-completed
Started
2024-10-23 14:31:01
ago
0 / 3 tasks completed
3 tasks still to be completed
Details
Required Tasks
| Task | Status |
|---|---|
[ y] Have you followed the guidelines in our Contributor's Guide (including the pre-commit check)? |
Incomplete |
| [ y] Have you checked to ensure there aren't other open Pull Requests for the same update/change? | Incomplete |
| Does your submission pass tests? -- Not sure | Incomplete |
| Have you lint your code locally prior to submission? -- Not sure | Incomplete |
| Have you added an explanation of what your changes do and why you'd like us to include them? -- In conversation, yes. | Incomplete |
| [y ] Have you successfully tested your changes locally? | Incomplete |
Can you please run pre-commit (and set it up to run on all future commits from your CUPiD directory)? Instructions are in the wiki (specifically, in the Contributor's Guide). I would recommend running both pre-commit run --all-files and pre-commit install from (cupid-dev), and then committing additional changes to the branch from that environment as well |
Incomplete |
There are a lot of empty cells in the notebook that can probably be removed. Also, adding the hide-input tag will hide the input cell when creating the JupyterBook webpage while hide-cell will hide both the input and output; you might want to add one of those or the other to each cell in your notebook to make the webpage a little easier to view |
Incomplete |
Can you please remove all the output from the notebook in examples/nblibrary/? Just a simple "restart kernel and clear output" before saving will keep it as a blank template. |
Incomplete |
I see a bunch of /glade/derecho/scratch/mdfowler/tmp/ipykernel_13813/1925081246.py:42: DeprecationWarning: Conversion of an array with ndim > 0 to a scalar is deprecated, and will error in future. Ensure you extract a single element from your array before performing this operation. (Deprecated NumPy 1.25.), and I think the warnings are pointing to lat_fluxnet[iStation] = siteInfoDF['LOCATION_LAT'].values[indStation] and lon_fluxnet[iStation] = siteInfoDF['LOCATION_LONG'].values[indStation] in [9]; can you figure out the correct indexing to make sure both sides of the equation are scalars and avoid this warning? (I'm happy to help you play with that if you'd like, though I won't have a chance until tomorrow) |
Incomplete |
Loading