Add WeatherFiles plugin 0.1.0.0-alpha1#1378
Conversation
Initial Alpha submission of the WeatherFiles plugin for OpenCPN. WeatherFiles browses 27+ European weather models, lets the user pick an area on the chart, and downloads sliced GRIB2 files that open directly in OpenCPN's built-in GRIB display. Backend at api.weatherfiles.com/v1. Targets covered in this submission: - debian-x86_64-12-bookworm - darwin-wx32-arm64-x86_64-14.3.1-macos (Universal, Developer ID signed + Apple-notarized) - msvc-x86-wx32-10.0.20348-MSVC - flatpak-x86_64-25.08-flatpak - flatpak-aarch64-25.08-flatpak Plugin source: https://github.com/bartmanuel/weatherfiles_pi Cloudsmith host: bartmanuel-fgsm/weatherfiles-alpha (public OSS plan)
|
Hi, if there is anything I can do to clarify or enhance this PR to help you process it let me know. Always ready to help get things moving! |
|
Friendly nudge on this one — it's my first OpenCPN submission, so I want to make sure I've routed it correctly rather than leave it sitting silently. @jongough — since you handle the A couple of things I think are just first-time-contributor friction rather than problems with the PR:
Plugin source: https://github.com/bartmanuel/weatherfiles_pi ( Glad to make any adjustments you'd like. Thanks for taking a look! |
|
First of all: this plugin looks interesting and I'd love to make test. Starting the workflows now. Please note there is a way to publish a plugin catalog which bypasses the need to make a PR to this repo. This is documented in TESTING.md EDIT: Users like me then needs to use the custom URL you have created to test. |
|
Ouch... only @bdbcat can start workflows here. |
|
BTW: the Flatpak targets are wrong. We can look into this later |
|
I walked through the steps to create a custom URL at https://raw.githubusercontent.com/leamas/plugins/refs/heads/bartmanuel/pr/ocpn-plugins.xml. However, your cloudsmith problems is a roadblock. Not that big one, though: when you create a cloudsmith repo there is an option to mark it as open-source. Such repos are free, no fees are involved. But then again, there is only one chance when creating the repo. See the developer manual Please update your repos, perhaps by deleting+ re-creating or just use some new names, and update this PR if need be. |
|
I could approve the workflow instantly. |
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Thanks @bdbcat for approving the run — and for the note that the workflow validates form only and that hosting isn't tied to Cloudsmith. Good to know. (Cloudsmith approved OSS status, so the tarball hosting is now solid again). The run surfaced a real issue: the
The new push re-triggered CI_Pull but it's back to "action_required" (first-time-contributor gate). Could you approve the run once more when you have a moment? It should be green now. Thanks again! |
|
I submitted a PR moving to the shipdriver template instead. This has all required builds in place besides android which needs some love related to curl. |
|
@bartmanuel : All of your issues are fixed in the pending PR, which also adds more useful builds. If you want to open a more informal communications channel many of us hangs around in Zulip, see the manual. Zulip is somewhat like Slack and easy to use. |
|
Sorry, I have been distracted with other plugins and have not responded. It appears Alec is handling this now. |
|
Yes, I can see that. Unfortunately shipdriver is a moving feast which seems to be re-written on a continual basis. FE2 is a stable build that updates without constant user modifications to make it work. But..... |
|
Testplugin is actually legacy. We (the project maintainers) do not recommend it for new plugins. |
|
The choice of build environment is totally up to you. |
|
I am not aware that FE2 (testplugin based) is 'legacy'. It works, is up to date with all build environments and is used by quite a few plugins. I am a project maintainer and recommend it for stability, ease of use, reduction in churn and 'it just works'. New versions of FE2 are released when needed and require only copying a few files which do not impact on customisations made by other developers. All changes for a plugin are maintained in the main CMakeLists.txt file. |
Having used both approaches (testplugin and shipdriver) I can highly recommend the latter, so yes, leave in it's current state (shipdriver) would be my recommendation too. |
|
OpenCPN core is agnostic to plugin template used. You are correct. Whatever method a dev would like to use to produce the tarballs and metadata is acceptable. |
|
@bdbcat ; Thanks for the confirmation. Is all clear for merging the PR then? |
|
Merged. |
|
thanks @bdbcat ! |
Summary
Initial Alpha submission of WeatherFiles — an OpenCPN plugin that browses 27+ European weather models, lets the user pick an area on the chart, and downloads sliced GRIB2 files that open directly in OpenCPN's built-in GRIB display.
Backend lives at https://api.weatherfiles.com/v1 (free tier + paid plan, documented at https://developers.weatherfiles.com).
Targets
5 metadata XMLs added, all pointing at a public Cloudsmith repo (
bartmanuel-fgsm/weatherfiles-alpha, OSS plan):weatherfiles_pi-0.1.0.0-debian-x86_64-12-bookworm.xmlweatherfiles_pi-0.1.0.0-darwin-wx32-arm64-x86_64-14.3.1-macos.xmlweatherfiles_pi-0.1.0.0-msvc-x86-wx32-10.0.20348-MSVC.xmlweatherfiles_pi-0.1.0.0-flatpak-x86_64-25.08-flatpak.xmlweatherfiles_pi-0.1.0.0-flatpak-aarch64-25.08-flatpak.xmlTesting
v0.1.0.0-alpha1on https://github.com/bartmanuel/weatherfiles_pi/.status: Accepted).api.weatherfiles.com/v1; no inbound sockets.Plugin source
https://github.com/bartmanuel/weatherfiles_pi —
mainis at tagv0.1.0.0-alpha1. The repo'sREADME.mdwalks through the 3-step download wizard, save-as-set, and the security model.Happy to make any adjustments the maintainers want — first OpenCPN submission for me.