Skip to content

Add mode zero, no wind shear - #30

Merged
ufechner7 merged 6 commits into
mainfrom
modezero
May 30, 2026
Merged

Add mode zero, no wind shear#30
ufechner7 merged 6 commits into
mainfrom
modezero

Conversation

@ufechner7

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a no-wind-shear / constant wind profile mode to the atmospheric wind profile handling and exposes it through a new example plot.

Changes:

  • Adds CONSTANT=0 to ProfileLaw and handles profile law 0 in calc_wind_factor.
  • Adds a no-wind-shear example script and includes it in the examples menu.
  • Qualifies the wind shear plotting call with ControlPlots.plot.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

File Description
src/AtmosphericModels.jl Adds the constant wind profile enum value and calculation branch.
examples/plot_windshear.jl Uses the explicit ControlPlots.plot call.
examples/plot_windshear_zero.jl Adds an example plotting constant wind speed with no shear.
examples/menu.jl Adds the new no-wind-shear plot example to the menu.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/AtmosphericModels.jl
Comment thread src/AtmosphericModels.jl
Comment thread src/AtmosphericModels.jl

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comment thread test/runtests.jl
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@ufechner7 ufechner7 linked an issue May 30, 2026 that may be closed by this pull request
@ufechner7 ufechner7 self-assigned this May 30, 2026
@ufechner7
ufechner7 merged commit 5096e28 into main May 30, 2026
7 checks passed
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.

Add uniform wind profile

2 participants