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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Type property pto sim #1064

Merged
merged 1 commit into from
Jul 10, 2023
Merged

Type property pto sim #1064

merged 1 commit into from
Jul 10, 2023

Conversation

jleonqu
Copy link
Contributor

@jleonqu jleonqu commented May 23, 2023

This PR changes the 'type' property definition for PTO-Sim blocks. Now the 'type' property is similar to the waveClass 'type' property.

This is the new 'type' property definition:

type: This property must be defined to specify the type of block that will be used. The type value of each block is presented below:

type = 'electricGen' ---- Electric generator equivalent circuit
type = 'hydraulicCyl' ---- Hydraulic cylinder
type = 'hydraulicAcc' ---- Hydraulic accumulator
type = 'rectCheckValve' ---- Rectifying check valve
type = 'hydraulicMotor' ---- Hydraulic motor
type = 'linCrank' ---- Linear crank
type = 'adjustableRod' ---- Adjustable rod
type = 'checkValve' ---- Check valve
type = 'ddLinearGen' ---- Direct drive linear generator
type = 'ddRotaryGen' ---- Direct drive Rotary generator

@jtgrasb
Copy link
Contributor

jtgrasb commented Jun 30, 2023

Just looked over this PR and the updates look good and running smoothly with the corresponding WEC-Sim Applications PR.

@kmruehl kmruehl added the PTO-Sim PTO-Sim (ptoSimClass.m) label Jul 10, 2023
@kmruehl
Copy link
Contributor

kmruehl commented Jul 10, 2023

@jleonqu this looks good to me. I'm going to merge it, along with WEC-Sim/WEC-Sim_Applications#33

@kmruehl kmruehl merged commit 636eed5 into WEC-Sim:dev Jul 10, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PTO-Sim PTO-Sim (ptoSimClass.m)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants