Skip to content

Python preprocessing image#1080

Draft
groutr wants to merge 42 commits intoNOAA-OWP:tifrom
groutr:python_preprocessing_image
Draft

Python preprocessing image#1080
groutr wants to merge 42 commits intoNOAA-OWP:tifrom
groutr:python_preprocessing_image

Conversation

@groutr
Copy link
Copy Markdown
Contributor

@groutr groutr commented Feb 6, 2025

Convert python-preprocessing lambda function from layer based to image based.

@groutr
Copy link
Copy Markdown
Contributor Author

groutr commented Feb 6, 2025

A small benchmark for max_values.

# Initial max_values
Duration
359703.73 ms
Billed duration
368240 ms
Resources configured
1024 MB
Max memory used
821 MB

# Using xarray+fsspec and h5netcdf to read remote files directly
# More memory efficient max value computation
Duration
156503.26 ms
Billed duration
156504 ms
Resources configured
6000 MB
Max memory used
763 MB

@groutr
Copy link
Copy Markdown
Contributor Author

groutr commented Feb 11, 2025

A small benchmark for anomoly

Duration
157696.26 ms
Billed duration
157697 ms
Resources configured
10240 MB
Max memory used
7439 MB

Duration
149726.80 ms
Billed duration
155115 ms
Resources configured
10240 MB
Max memory used
2877 MB

Significant reduction in memory usage.

shawncrawley
shawncrawley previously approved these changes Feb 11, 2025
@shawncrawley shawncrawley dismissed their stale review February 11, 2025 18:23

Didn't realize the PR is a draft

@groutr
Copy link
Copy Markdown
Contributor Author

groutr commented Feb 25, 2025

This PR is becoming to large to effectively review. Therefore, I'm splitting into smaller PRs.

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