Skip to content
New issue

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

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

Already on GitHub? # to your account

New environment variable #432

Merged
merged 8 commits into from
Jul 20, 2020
Merged

New environment variable #432

merged 8 commits into from
Jul 20, 2020

Conversation

rs028
Copy link
Collaborator

@rs028 rs028 commented Jul 10, 2020

Create a new environment variable ASA (aerosol surface area). It should work in the same way as BLHEIGHT, e.g., can be NOTUSED (default), CONSTRAINED or given a fixed value, and it can be used to calculate rate coefficients. For example:

% ASA*525.068*(TEMP@0.5)*2.0D-3   :   O3   =   ;

@rs028
Copy link
Collaborator Author

rs028 commented Jul 10, 2020

@spco Apart from the tests failing, which I will fix later, is this all there is to do to add a new environment variable?

@spco
Copy link
Collaborator

spco commented Jul 10, 2020

Looks about so from searching the codebase. You'll also need to update

subroutine update_p(p, q, TEMP, N2, O2, M, RH, H2O, DEC, BLHEIGHT, DILUTE, JFAC, ROOFOPEN, J, RO2) bind(c,name='update_p')
to match new signature, and add ASA to reservedSpeciesList at
reservedSpeciesList = ['N2', 'O2', 'M', 'RH', 'H2O', 'DEC', 'BLHEIGHT', 'DILUTE', 'JFAC', 'ROOF', 'RO2']
. I can't guarantee that that's everything, but looks roughly correct.

1 similar comment
@spco
Copy link
Collaborator

spco commented Jul 10, 2020

Looks about so from searching the codebase. You'll also need to update

subroutine update_p(p, q, TEMP, N2, O2, M, RH, H2O, DEC, BLHEIGHT, DILUTE, JFAC, ROOFOPEN, J, RO2) bind(c,name='update_p')
to match new signature, and add ASA to reservedSpeciesList at
reservedSpeciesList = ['N2', 'O2', 'M', 'RH', 'H2O', 'DEC', 'BLHEIGHT', 'DILUTE', 'JFAC', 'ROOF', 'RO2']
. I can't guarantee that that's everything, but looks roughly correct.

@codecov
Copy link

codecov bot commented Jul 20, 2020

Codecov Report

Merging #432 into master will increase coverage by 0.00%.
The diff coverage is 80.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #432   +/-   ##
=======================================
  Coverage   88.72%   88.73%           
=======================================
  Files          17       17           
  Lines        2262     2264    +2     
=======================================
+ Hits         2007     2009    +2     
  Misses        255      255           
Flag Coverage Δ
#build 62.49% <80.00%> (+0.04%) ⬆️
#tests 87.89% <80.00%> (+0.01%) ⬆️
#unittests 35.73% <0.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/inputFunctions.f90 86.31% <ø> (ø)
src/constraintFunctions.f90 81.20% <50.00%> (+0.12%) ⬆️
src/solverFunctions.f90 91.25% <100.00%> (+0.11%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 018cf02...3f39361. Read the comment docs.

@rs028
Copy link
Collaborator Author

rs028 commented Jul 20, 2020

If there are no more comments I think this can be merged.

@rs028 rs028 merged commit 2e26171 into AtChem:master Jul 20, 2020
@rs028 rs028 deleted the add_envvar branch July 20, 2020 18:14
@rs028 rs028 mentioned this pull request Feb 21, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants