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

Derived data with polar calculations. #7

Closed
wants to merge 6 commits into from
Closed

Derived data with polar calculations. #7

wants to merge 6 commits into from

Conversation

ieb
Copy link
Contributor

@ieb ieb commented Sep 5, 2017

This pull request adds polar performance calculations. It needs review since some of the keys may not be 100% correct, however it seems to work for me and flows through to other modules. At the moment the polar data is in javascript, which may not be suitable for everyone.

Doing the pull request as it was asked for at https://groups.google.com/forum/#!topic/signalk/YpmDpfv-Ro4

@sbender9
Copy link
Member

sbender9 commented Sep 8, 2017

I'm sorry @ieb , but there are too many things combined into to this one pull request. Many not related at all to the title of the PR. Do you have time to separate this into well defined PR's?

@sbender9
Copy link
Member

sbender9 commented Sep 8, 2017

We can't include the polar data in the plugin code. It would need to come from somewhere else.

@sbender9
Copy link
Member

sbender9 commented Sep 8, 2017

@joabakk aren't you doing things with polar data? Are there conflicts here or points of integration?

@tkurki
Copy link
Member

tkurki commented Sep 8, 2017

Well, now that we have some code that uses polar tables it is time to get polar data representation in the model in shape and create a webapp/plugin combo that allows you to enter, edit and visualise polar data and use it. Sounds awfully much like a signalk-polars plugin to me, based on work here and discussions at SignalK/specification#360.

@ieb
Copy link
Contributor Author

ieb commented Sep 8, 2017

@sbender9 I agree, although to get the performance calcs working on my boat I needed everything other than cogMagnetic. I needed that because the COG sentence needed it.
The polar representation is pretty standard as used in most other systems, Expedition, NKE, SailGribWR. It might be better in CSV form if you want users to be able to edit it, but putting CSV into json (which is almost what I had) is relatively trivial.
In most routing I have done, I have dealt with sea state separately as it's too hard predict sea state as that depends on the weather model for 1000s of miles around, not just the current wind, and its highly unpredictable.
Since someone else is already working on performance calculations it doesnt make sense to merge this pull request. TBH, I am not 100% certain everything is correct, and I still get results out a sea I dont always believe when it comes to the optimal angles.

@tkurki without the output of the derived data produced from the performance calculation the PNKE wont work, and some of the ones in the TODO area might not work at all. You might want to drop those from the NMEA0183 pull request I did as they are bound to generate unnecessary noise from anyone wanting to drive the NKE Android app for real.

I will keep developing and correcting the branch in the repo where this came from. Thank you all for the pointer to derived data and other help.

@ieb ieb closed this Sep 8, 2017
@joabakk
Copy link
Contributor

joabakk commented Sep 8, 2017

I have made a plugin and webapp pair for polar diagrams. This predates derived-data and SignalK/specification#359. The plugin (https://github.com/joabakk/signalk-polar) does the calculations, submits the dynamic values that exist(ed) in the schema to the server and stores the polar table in mysql. The webapp , https://github.com/joabakk/signalk-polar-graphing picks this table up and (as far as I recall) presents the diagram via html.
It may or may not make sense to put some of the polar calculations into derived-data. Don't give up just yes @ieb!

@ieb
Copy link
Contributor Author

ieb commented Sep 8, 2017 via email

@joabakk
Copy link
Contributor

joabakk commented Sep 8, 2017

Eh licence? It's on npm. Reading more thoroughly "The Highsoft Standard Terms and Conditions applies if you run a commercial or governmental website, intranet or project". The way I read it, we are good

@ieb
Copy link
Contributor Author

ieb commented Sep 8, 2017 via email

@joabakk
Copy link
Contributor

joabakk commented Sep 22, 2017

Thanks @ieb for the pointer. I have signed up and informed as they require now

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 this pull request may close these issues.

4 participants