Changes

Fixed Wb_flows discretization terms for DynamicPipe

- Fixes made for all structure cases: av_b, a_vb, a_v_b, and av_vb
- The new discretizations terms give more accurate derivatives associated with the volumes
- Energy is now properly conserved (previously av_b, a_vb, and a_v_b had conservation errors)

Reduce guess for initial temperature difference across wall.

The reason is two-fold:
* It is an order of magnitude too large.
* In some configurations it can cause simulation problems.

Remove unit errors.
This is related to #3881
These changes are completely safe.

Avoid unit error.
Adding a protected element in an example model should be ok.

Fix incorrect start-value.
Detected due to unit-checking.

Give a proper definition of machine constants.
They are compatible with C, and usable.
Closes #2056

Update description in ModelicaServices as well.

And values.

Rename record instance names ????Data => ????Data3

Fix final parameter propagation
so that m = 3 is not propagated to the core parameter records

Fix final in parameter record of FundamentalWave

Unify wording of docstrings of parameter records

Unit correctness.

Remove incorrect unit-casting.

Implement option 4; that is using a reference impedance with description.

Put reference impedance into partial induction machine model

Apply ZsRef also to induction machine models

Apply changes to Modelica.Magnetic.FundamentalWave

Apply changes to Modelica.Magnetic.QuasiStaic.FundamentalWave

Add DynamicPipe energy conservation tests to ModelicaTest

Updated tolerance for energy conservation checks

Improve indentation of html code

Html code: No indentation within 'address' tag

Fix non-matching 'tr' tag and delete '>' added by mistake

Delete multiple and/or leading whitespaces

Use prefix style 'quantity' for FixedBoundary.X

This follows the style in Modelica.Media.Interfaces.PartialMedium.MassFlowRate and gives tools a chance of making sense of the quantity.

Remove modification of 'quantity'

Remove empty modifications

Fix unit error in RobotR3.OneAxis

Fix unit errors in ArmatureStroke

Fix unit error in CCCVcharging

Modifying min attribute of gain to 1

removing min attribute in gain

Use SI.Radius instead of SI.Length

ReferenceMoistAir, corrected error in function pd_pTX_der (issue #3666)

Remove plural 's' in 'height of ramps'

Assert statement to avoid zero gain in controller and improved description string for controller gain

Changed to full path for consistency

Removing unintended code modifications

Delete unused variable 'dens' and corresponding 'R_air'

Delete gasForce.dens from reference comparison signals

Replace 'Inline=false' with late inlining
See  #3908

Unify spelling of controllers with dash omitted

Dash omitted to unify spelling of controllers across library.

Revert "Unify spelling of controllers with dash omitted"

This reverts commit 9bcd99f1d2ca81d20b67008b4b22cf2135cbac3d.

unify spelling of controllers

unify spelling of controllers by omitting dash in description and documentation

Rename RiCurrent -> Isc

Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>

Adapt to renamed variable

Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>

Make it public, as suggested

Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>

#4107 missing commits in master (#4140)

* Resolved issue #4072 (for Modelica/Electrical)

* Resolved issue #4072 (for rest of Modelica)

* Squash doubled point {0,-50.5}

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Fix near-zero coordinate in connection's annotation

* Merge branch 'DagBruck_01' of https://github.com/modelica/ModelicaStandardLibrary into DagBruck_01

---------

Co-authored-by: Dag Brück <dag.brueck@3ds.com>
Co-authored-by: tobolar <tobolar@users.noreply.github.com>
Co-authored-by: tobolar <jakub.tobolar@dlr.de>

Update codespell exceptions (#4143)

Bump actions to avoid deprecation warnings (#4120)

Amendment for #4140: Fix merge conflict (#4146)

lower case for controller in Modelica/Blocks/Continuous.mo

Co-authored-by: tobolar <tobolar@users.noreply.github.com>

lower case for controller in Modelica/Clocked/Examples/Systems/ControlledMixingUnit.mo

Co-authored-by: tobolar <tobolar@users.noreply.github.com>

unify spelling of controllers

unify spelling of controllers and lower case.

Fix false calculation of w (omega) when resolveInFrame == frame_resolve

And add unit for M

Add rotation about x-axis to enable a spatial motion

Have a flag for avoiding unnecssary discontinuities.
Closes #3548

Add smooth.

ExpSine is more complicated.

oops

Fix syntax.

Simplify the mean formula.
Closes #3906

Add startTime as well.

Restore the use of t_eps just in case, but allow setting it to zero.

And update description.

Avoid smoothness.

Update Modelica/Blocks/Sources.mo

Co-authored-by: Henrik Tidefelt <henrikt@wolfram.com>

Update Modelica/Blocks/Sources.mo

Co-authored-by: Henrik Tidefelt <henrikt@wolfram.com>

Rephrase

Simpler.

If the limits don't make sense use linear interpolation.
See https://github.com/modelica/ModelicaStandardLibrary/issues/4128#issuecomment-1589465543

Fix typo

Co-authored-by: Thomas Beutlich <modelica@tbeu.de>

Omit modifiers of animation parameters

Documentation and description for FluidHeatFlow library improved (#4141)

* Improved doc strings and documentation

* Improved documentation Updated

* Update ParallelCooling

* Lower case spelling of coolant

* Corrected misspelling of coolant in documentation of IndirectCooling.mo

Corrected misspelling of coolant in two instances of documentation

---------

Co-authored-by: Christian Kral <dr.christian.kral@gmail.com>
Co-authored-by: arunkumar-narasimhan <124154466+arunkumar-narasimhan@users.noreply.github.com>

Update Modelica/Blocks/Sources.mo

Co-authored-by: Henrik Tidefelt <henrikt@wolfram.com>

Remove unit-casting

Remove all uses of unitFrequency. That was previously necessary when Gain.k had unit 1

Add Evaluate=true.

Added units to mathematical constants (#4155)

* Added units to mathematical constants

Delete redundant points of polygon annotation of 'Spice3' connectors (#4135)

Fix unit for mu0.
This corrects an error introduced by #4155

Simpler.

Removed out-of-date/incorrect comment about epsilon_0

Fix codespell exceptions (CI) (#4165)

* Fix typos
* Fix codespell exceptions

Delete successful and skipped workflow runs (#4177)

refs #4134

Meke internally used ports of optional housing port and optional thermal port protected

Update annotations to be correct.
Fixes #4180

Update Modelica/Magnetic/QuasiStatic/FundamentalWave/Examples/BasicMachines/SynchronousMachines/SMR_CurrentSource.mo

Apply suggestions from code review

Revert

refs #4180 Fix Text annotation

Update Modelica/Magnetic/QuasiStatic/FluxTubes/package.mo

Co-authored-by: tobolar <tobolar@users.noreply.github.com>

Restore documentation (#4184)

Fix oversight of f2eec0eff074fe7e20a13593f8c9e07b8a20dd6f.

Fix regex to optionally ignore quoted strings (#4183)

Previously, a Text.lineColor annotation was not detected, if there was a closing parenthesis (within a quoted Text.textString annotation), e.g.

```
Text(extent={{-100,80},{-80,60}}, textString="1)", lineColor={0,0,255})
```

Fix unit errors in ExternalCombiTimeTable constructor (#4188)

* Fix unit errors in ExternalCombiTimeTable constructor

The errors are in relation to how the constructor is called in Modelica.Blocks.Sources.CombiTimeTable.

* Don't assume use of scaled time

Remove use of unknown variable useReset

Remove unknown parameter multiColoredSurface. Note that the actual surface has =false for it.

The parameter 'target' does not exist.

Correct use of incorrect parameter and enumeration. However, the basic design doesn't work.

Remove unknown parameter useMeanDensity

KC and DP_Fric do not exist in this class

Cleanup

As indicated in #4046 it we need to revert part of #4155 since it is causing too many regressions for a minor release.
With proposed unit-checking in Dymola this reduces the warnings from 2301 to 1224 (Most - but not all are unit-related.)

Obviously many of them could be corrected, but I don't see that we have the resources to focus on that for this minor release.
The first models found were CompareTransformers, SeriesResonance in Modelica.Electrical.Analog.Examples.

Correct units for normalizing vectors.

Fix link to Holman2010 (#4199)

Fix and add ITEA links (#4196)

Fix broken links (#4202)

Remove public

Coordination of #4101: Fix unit errors
in other Batteries.Examples

Fix instance names

Close #4101: Fix unit error in a single PR

Remove blanks

Remove redundant diagram text for most Analog examples

Remove diagram text of CompareTransformers

Fix factor 2 in calculation

Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>

Update .mailmap file

Create by command

```
git log --raw | grep "^Author: " | sort | uniq | cut -d ' ' -f2- | sed 's/^//' > .mailmap
```

as reported by https://kevin.burke.dev/kevin/easy-maintenance-of-your-authors-file/

Remove duplicates in SignalNames of 10 examples of ModelicaTest (#4179)

See https://github.com/modelica/MAP-LIB_ReferenceResults/pull/6

Co-authored-by: Matthias Schaefer <matthias.schaefer@ltx.de>

Fix codespell exceptions (#4203)

* Fix typos
* Fix codespell exceptions

Remove real relation in non-discrete-time controlled if-statement.

Remove unnecessary noEvent

Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>

Fix broken link (#4211)

Use 1.14 instead of sqrt(1.3)

Acting on thumbs up only for this change in PR discussion.

Update matio library to 1.5.24 (#4222)

Add all defined outputs of the test example into reference signals

Initialize output array C even when multiColoredSurface=false

Set modifier of 'C' being 'final'

Documentation: modifier 'C' shall be 'final'

refs #4174: Fix (and rename) parameters (#4176)

Fixed Wb_flows discretization terms for DynamicPipe

- Fixes made for all structure cases: av_b, a_vb, a_v_b, and av_vb
- The new discretizations terms give more accurate derivatives associated with the volumes
- Energy is now properly conserved (previously av_b, a_vb, and a_v_b had conservation errors)

Reduce guess for initial temperature difference across wall.

The reason is two-fold:
* It is an order of magnitude too large.
* In some configurations it can cause simulation problems.

Remove unit errors.
This is related to #3881
These changes are completely safe.

Avoid unit error.
Adding a protected element in an example model should be ok.

Fix incorrect start-value.
Detected due to unit-checking.

Give a proper definition of machine constants.
They are compatible with C, and usable.
Closes #2056

Update description in ModelicaServices as well.

And values.

Rename record instance names ????Data => ????Data3

Fix final parameter propagation
so that m = 3 is not propagated to the core parameter records

Fix final in parameter record of FundamentalWave

Unify wording of docstrings of parameter records

Unit correctness.

Remove incorrect unit-casting.

Implement option 4; that is using a reference impedance with description.

Put reference impedance into partial induction machine model

Apply ZsRef also to induction machine models

Apply changes to Modelica.Magnetic.FundamentalWave

Apply changes to Modelica.Magnetic.QuasiStaic.FundamentalWave

Add DynamicPipe energy conservation tests to ModelicaTest

Updated tolerance for energy conservation checks

Improve indentation of html code

Html code: No indentation within 'address' tag

Fix non-matching 'tr' tag and delete '>' added by mistake

Delete multiple and/or leading whitespaces

Use prefix style 'quantity' for FixedBoundary.X

This follows the style in Modelica.Media.Interfaces.PartialMedium.MassFlowRate and gives tools a chance of making sense of the quantity.

Remove modification of 'quantity'

Remove empty modifications

Fix unit error in RobotR3.OneAxis

Fix unit errors in ArmatureStroke

Fix unit error in CCCVcharging

Modifying min attribute of gain to 1

removing min attribute in gain

Use SI.Radius instead of SI.Length

ReferenceMoistAir, corrected error in function pd_pTX_der (issue #3666)

Remove plural 's' in 'height of ramps'

Assert statement to avoid zero gain in controller and improved description string for controller gain

Changed to full path for consistency

Removing unintended code modifications

Delete unused variable 'dens' and corresponding 'R_air'

Delete gasForce.dens from reference comparison signals

Replace 'Inline=false' with late inlining
See  #3908

Unify spelling of controllers with dash omitted

Dash omitted to unify spelling of controllers across library.

Revert "Unify spelling of controllers with dash omitted"

This reverts commit 9bcd99f1d2ca81d20b67008b4b22cf2135cbac3d.

unify spelling of controllers

unify spelling of controllers by omitting dash in description and documentation

Rename RiCurrent -> Isc

Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>

Adapt to renamed variable

Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>

Make it public, as suggested

Co-authored-by: Hans Olsson <HansOlsson@users.noreply.github.com>

#4107 missing commits in master (#4140)

* Resolved issue #4072 (for Modelica/Electrical)

* Resolved issue #4072 (for rest of Modelica)

* Squash doubled point {0,-50.5}

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Delete doubled connection's coordinate

* Fix near-zero coordinate in connection's annotation

* Merge branch 'DagBruck_01' of https://github.com/modelica/ModelicaStandardLibrary into DagBruck_01

---------

Co-authored-by: Dag Brück <dag.brueck@3ds.com>
Co-authored-by: tobolar <tobolar@users.noreply.github.com>
Co-authored-by: tobolar <jakub.tobolar@dlr.de>

Update codespell exceptions (#4143)

Bump actions to avoid deprecation warnings (#4120)

Amendment for #4140: Fix merge conflict (#4146)

lower case for controller in Modelica/Blocks/Continuous.mo

Co-authored-by: tobolar <tobolar@users.noreply.github.com>

lower case for controller in Modelica/Clocked/Examples/Systems/ControlledMixingUnit.mo

Co-authored-by: tobolar <tobolar@users.noreply.github.com>

unify spelling of controllers

unify spelling of controllers and lower case.

Fix false calculation of w (omega) when resolveInFrame == frame_resolve

And add unit for M

Add rotation about x-axis to enable a spatial motion

Have a flag for avoiding unnecssary discontinuities.
Closes #3548

Add smooth.

ExpSine is more complicated.

oops

Fix syntax.

Simplify the mean formula.
Closes #3906

Add startTime as well.

Restore the use of t_eps just in case, but allow setting it to zero.

And update description.

Avoid smoothness.

Update Modelica/Blocks/Sources.mo

Co-authored-by: Henrik Tidefelt <henrikt@wolfram.com>

Update Modelica/Blocks/Sources.mo

Co-authored-by: Henrik Tidefelt <henrikt@wolfram.com>

Rephrase

Simpler.

If the limits don't make sense use linear interpolation.
See https://github.com/modelica/ModelicaStandardLibrary/issues/4128#issuecomment-1589465543

Fix typo

Co-authored-by: Thomas Beutlich <modelica@tbeu.de>

Omit modifiers of animation parameters

Documentation and description for FluidHeatFlow library improved (#4141)

* Improved doc strings and documentation

* Improved documentation Updated

* Update ParallelCooling

* Lower case spelling of coolant

* Corrected misspelling of coolant in documentation of IndirectCooling.mo

Corrected misspelling of coolant in two instances of documentation

---------

Co-authored-by: Christian Kral <dr.christian.kral@gmail.com>
Co-authored-by: arunkumar-narasimhan <124154466+arunkumar-narasimhan@users.noreply.github.com>

Update Modelica/Blocks/Sources.mo

Co-authored-by: Henrik Tidefelt <henrikt@wolfram.com>

Remove unit-casting

Remove all uses of unitFrequency. That was previously necessary when Gain.k had unit 1