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

Support for shaft rpm and remote control throttle position #593

Open
pbegg opened this issue Nov 12, 2020 · 3 comments · May be fixed by #594
Open

Support for shaft rpm and remote control throttle position #593

pbegg opened this issue Nov 12, 2020 · 3 comments · May be fixed by #594

Comments

@pbegg
Copy link

pbegg commented Nov 12, 2020

Working with some voyage data recorders and a couple of extra paths I think could be added that would be useful.

/vessels/<RegExp>/propulsion/<RegExp>/shaft/revolutions
Units: Hz (Hertz)
Description: Shaft revolutions (x60 for RPM)

It would be useful to have a positive and negative value for shaft direction (CW or CCW)


/vessels/<RegExp>/propulsion/<RegExp>/remoteControl/throttlePosition
Units: Ratio -1 to 1
Description: Throttle Position, -1<=ratio<=1, 1 is 100%


Shaft rpm is helpful for determining whether or not the gear box has engaged. An example is a master is coming into a wharf and goes to select astern, the throttlePosition shows that the lever has been moved to the astern position but shaft rpm is still showing a positive (or 0) value and astern gear has not been engaged.

What are everyone's thoughts

@tkurki
Copy link
Member

tkurki commented Nov 12, 2020

Just a quick comment: existing similar paths are revolutions - rpm in Hz is a bit of a wtf...

@pbegg
Copy link
Author

pbegg commented Nov 12, 2020

That was a copy and paste mistake. I agree and have edited the original post.

@hwater
Copy link

hwater commented Apr 6, 2021

But don´t forget this, even if it is called RPM:
It would be useful to have a positive and negative value for shaft direction (CW or CCW)

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 a pull request may close this issue.

3 participants