Skip to content

Implementation of Gaussian pulse signal source - #4017

Open
AHaumer wants to merge 10 commits into
modelica:masterfrom
AHaumer:GaussianPulse
Open

AHaumer wants to merge 10 commits into
modelica:masterfrom
AHaumer:GaussianPulse

Conversation

@AHaumer

@AHaumer AHaumer commented Jul 31, 2022

Copy link
Copy Markdown
Contributor

When preparing lectures for this fall, I noticed that an important signal source is missing: Gaussian pulse.

@AHaumer AHaumer added enhancement New feature or enhancement L: Blocks Issue addresses Modelica.Blocks labels Jul 31, 2022
@AHaumer AHaumer self-assigned this Jul 31, 2022
@dietmarw
dietmarw removed their request for review September 5, 2022 12:59
@beutlich
beutlich removed their request for review September 24, 2022 10:20
@christiankral

Copy link
Copy Markdown
Contributor

OK, as I have no actual application of a Gaussian pulse source, so I just want to summarize and clarify its behavior:

  • There is the option to create a sequence of pulses using the parameter nperiod
  • The pulses are not superimposed. Instead a strictly periodic signal with no overlap of the individual pulses is created.
  • So the behavior is different from creating a vector of single pulse sources with consecutive startTime and adding their outputs.

@AHaumer I suppose this behavior is intended, isn't it?

Comment thread Modelica/Blocks/Sources.mo Outdated
Comment thread Modelica/Blocks/Sources.mo Outdated
Comment thread Modelica/Blocks/package.mo Outdated
Comment thread Modelica/Blocks/Sources.mo Outdated
@AHaumer
AHaumer requested a review from tobolar September 10, 2026 15:35
Comment thread Modelica/Blocks/Sources.mo Outdated
Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>
@AHaumer
AHaumer requested a review from HansOlsson September 11, 2026 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or enhancement L: Blocks Issue addresses Modelica.Blocks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants