Skip to content

DT-459: extract sites config - #28

Open
janvt wants to merge 13 commits into
mainfrom
feat/DT-459-extract-sites-config
Open

DT-459: extract sites config#28
janvt wants to merge 13 commits into
mainfrom
feat/DT-459-extract-sites-config

Conversation

@janvt

@janvt janvt commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@janvt
janvt requested a review from victoria-cherkas August 7, 2026 10:03
Comment thread templates/COMMAND.j2

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks bigger than before? Is all of this correct?

Comment thread utils/flexpart_ifs_utils/model.py Outdated
Comment thread templates/COMMAND.j2 Outdated
LINIT_COND= 0, ! Output sensitivity to initial conditions (bkw mode only) [0]off 1]conc 2]mmr
SURF_ONLY= 0, ! Output only for the lowest model layer, used w/ LINIT_COND=1 or 2
CBLFLAG= 0, ! Skewed, not Gaussian turbulence in the convective ABL, need large CTL and IFINE
OHFIELDS_PATH= "../../flexin/", ! Default path for OH file

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apparently "./" is the default anyway (in code). And this line is unused for this container as it never reads any "OH climatology"? fields. This is what I was asking here, about this template being a lot larger. I think Claude just pulled a large template from source code, but as far as I can tell, we're still just rendering the lines we need (that's what the tests say).

Comment thread utils/flexpart_ifs_utils/model.py Outdated
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