Hydraulic docstrings updated to match code

[matthew-kapp]

Checklist

• Appropriate tests were added | N/A - only docstrings updated
• Any code changes were done in a way that does not break public API
• All documentation related to code changes were updated
• The new code follows the
  contributor guidelines, in particular the SciML Style Guide and
  COLPRAC.
• Any new documentation only uses public API

Additional context

• updated docstrings in utils.jl to remove P_int (in docstrings)
• updated docstrings in sources.jl to remove P_int (in docstrings)
• updated docstrings in components.jl to remove P_int, P_a_int, etc (in docstrings)
• solves issue No argument 'p_int' for HydraulicPort #354

[matthew-kapp]

There were quite a few errors when I ran docs/make, unrelated to my changes. Is this relevant i.e. do these errors need to be fixed first?

[ChrisRackauckas]

Doesn't the macro generate inits for each of the variables?

[matthew-kapp]

I don't know, so is it correct the way it is?

[ChrisRackauckas]

@ven-k ?

[ven-k]

Hydraulic library was not written with @mtkmodel. So the p_inits were explicitly added and later removed from the functions.
And the doc changes look fine.

---

The @mtkmodel doesn't automatically add inits (it creates a kwarg with var = ModelingToolkit.NoValue() value). Instead users should explicitly add that.

julia> @macroexpand1 @mtkmodel A begin
           @variables begin
               v
           end
       end
:(A = (ModelingToolkit.Model)($(Expr(:(=), :(__A__(; name,
v::Union{Nothing, Missing, ModelingToolkit.NoValue, Real} = ModelingToolkit.NoValue())), quote
...
...

[matthew-kapp]

Apologies for my Github clumsiness

@ChrisRackauckas @ven-k If the updates are OK, could you please approve PR #367 once it is ready?

[ChrisRackauckas]

This was closed?

[matthew-kapp]

@ChrisRackauckas @ven-k these updates were combined with PR #363

[matthew-kapp]

And I closed it by mistake