-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add the fetcher functionality and documentation on rtd #198
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #198 +/- ##
==========================================
+ Coverage 91.30% 91.72% +0.42%
==========================================
Files 26 28 +2
Lines 5437 5666 +229
==========================================
+ Hits 4964 5197 +233
+ Misses 473 469 -4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really nice to me! Thanks. Only one tiny correction.
But would be good to have some more eyes on it. @TAlonglong or @pnuu ?
Co-authored-by: Adam Dybbroe <adybbroe@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had a quick look
LGTM, but I need to run it and test it a bit before I can see if it is doing what it is supposed to. Then eventually we can iterate on it.
port: 1979 | ||
subscriber_config: | ||
addresses: | ||
- ipc://some_pipe |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this "inter-process communication" protocol? I didn't know Posttroll supported other than tcp://
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, tcp is hardcoded here and there in posttroll, but when passing addresses directly anything zeromq supports should work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR adds a pytroll-fetcher script and functionality to download files received through messages, and documentations pages for trollmoves to be hosted on readthedocs.