Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

PP saving soil depths incorrectly setting LBLEV=0 #4082

Closed
trexfeathers opened this issue Apr 7, 2021 · 5 comments
Closed

PP saving soil depths incorrectly setting LBLEV=0 #4082

trexfeathers opened this issue Apr 7, 2021 · 5 comments

Comments

@trexfeathers
Copy link
Contributor

馃悰 Bug Report

Expected behaviour

PP saving Single depth level (soil depth) should be setting pp.lblev correctly per level, rather than the current default of 0.

Screenshots

Environment

  • Iris Version: known for v2.3 onwards, but likely historic too
@github-actions
Copy link
Contributor

In order to maintain a backlog of relevant issues, we automatically label them as stale after 500 days of inactivity.

If this issue is still important to you, then please comment on this issue and the stale label will be removed.

Otherwise this issue will be automatically closed in 28 days time.

@github-actions github-actions bot added the Stale A stale issue/pull-request label Oct 29, 2023
Copy link
Contributor

This stale issue has been automatically closed due to a lack of community activity.

If you still care about this issue, then please either:

  • Re-open this issue, if you have sufficient permissions, or
  • Add a comment stating that this is still relevant and someone will re-open it on your behalf.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Nov 26, 2023
@bjlittle bjlittle removed the Stale A stale issue/pull-request label Nov 29, 2023
@bjlittle bjlittle reopened this Nov 29, 2023
@ESadek-MO ESadek-MO self-assigned this Nov 29, 2023
@ESadek-MO
Copy link
Contributor

Related to #3820

@ESadek-MO
Copy link
Contributor

So, after a bit of digging it seems pp.lblev can't be calculated within the _vertical_rules. There is the potential that there might be identifiers after this stage which would allow pp.lblev to be set, but I'm yet to find a straightforward solution to this.

@ESadek-MO
Copy link
Contributor

ESadek-MO commented Feb 7, 2024

@SciTools/peloton think that the solution to this is to include user-side workarounds in the Further Topics documentation.

@ESadek-MO ESadek-MO closed this as not planned Won't fix, can't repro, duplicate, stale Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants