Skip to content

Add minimal modal run to model dry deposition - #211

Merged
jcurtis2 merged 64 commits into
compdyn:masterfrom
zdaq12:final_modal
Aug 11, 2026
Merged

jcurtis2 merged 64 commits into
compdyn:masterfrom
zdaq12:final_modal

Conversation

@zdaq12

@zdaq12 zdaq12 commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@slayoo

slayoo commented Sep 16, 2025

Copy link
Copy Markdown
Collaborator

closing and reopening to trigger codecov analysis

@slayoo slayoo closed this Sep 16, 2025
@slayoo slayoo reopened this Sep 16, 2025
@codecov

codecov Bot commented Sep 16, 2025 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.29412% with 33 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.49%. Comparing base (e6356e7) to head (544d992).
⚠️ Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
src/scenario.F90 87.58% 19 Missing ⚠️
src/run_modal.F90 89.24% 10 Missing ⚠️
src/output.F90 93.02% 3 Missing ⚠️
src/spec_file.F90 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #211      +/-   ##
==========================================
+ Coverage   77.45%   78.49%   +1.04%     
==========================================
  Files          55       56       +1     
  Lines        9420     9755     +335     
==========================================
+ Hits         7296     7657     +361     
+ Misses       2124     2098      -26     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@zdaq12
zdaq12 requested a review from jcurtis2 June 13, 2026 14:15
Comment thread scenarios/7_drydep/2_process_modal.sh Outdated
Comment thread scenarios/7_drydep/1_run_modal.sh Outdated
Comment thread src/aero_dist.F90 Outdated
Comment thread src/aero_dist.F90 Outdated
Comment thread src/aero_dist.F90 Outdated
Comment thread src/scenario.F90 Outdated
Comment thread src/scenario.F90 Outdated
Comment thread src/scenario.F90 Outdated
Comment thread src/scenario.F90 Outdated
Comment thread src/output.F90 Outdated
zdaq12 and others added 20 commits June 24, 2026 21:12
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Co-authored-by: Jeffrey Curtis <jcurtis2@illinois.edu>
Removed redundant lines and cleaned up the integration logic for dry deposition loss rate calculation.
@zdaq12
zdaq12 requested a review from jcurtis2 July 18, 2026 18:23
Comment thread test/loss/test_loss_14.sh Outdated
@jcurtis2
jcurtis2 merged commit 7984020 into compdyn:master Aug 11, 2026
7 checks passed
jcurtis2 pushed a commit that referenced this pull request Oct 14, 2026
Extends the size-resolved dry deposition scheme from #211 to run_sect, so sectional, modal and particle-resolved runs can be compared directly.

- Sectional drydep: per-bin first-order loss at the bin-center deposition velocity (single aerosol species only).
- New optional do_aero_dilution (default yes, requires drydep). Setting it to no skips both background exchange and mixing-height dilution, so deposition can be isolated under a time-varying mixing height.
- Coagulation can now be disabled for sectional runs.
- Currently unsupported loss function constant/volume/chamber now abort in sectional and modal runs instead of silently doing nothing.
- Physics change: the deposition velocity is now V_d = V_s + 1/(R_a + R_s) replacing V_s + 1/(R_a + R_s + R_a R_s V_s)
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