-
Notifications
You must be signed in to change notification settings - Fork 80
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
Improvement on file spliting #16
Conversation
Allow an input file to be split into each individual message (if nchunk=-1)
Thank you. Have you included tests? |
Added the third test for nchunk=-1. OUTPUT_FILENAME_FORMAT changed to 3 digits, which led to file name changes in this script.
Thank you for the test. By the way have you signed the Contributor License Agreement? |
Hi chaps. Can I suggest you switch the osx_image in .travis.yml to xcode10.2? That may fix the MacOS build issue. |
Thanks Daniel. I will change it and push to develop |
I have not signed the CLA. I have tried a few links and it did not work. Please advise how to or email me a form that I can sign and send back to you. Thanks. |
Hi @wqu-bom, Do you see the agreement (after pressing the "CLA not signed yet" button) and able to sign it? PS: You might want to have a look at https://confluence.ecmwf.int/display/SUP/ECMWF+software+on+GitHub |
Tried the "CLA not signed yet" button, but it did not work (error msg below), nor the link "Details" above. |
Hi @wqu-bom, our bot was probably blocked by a firewall somewhere causing the error you received above -- although I could not replicate it. The bot is now on listening on port 80 so it should be accessible. If you wouldn't mind, please try again at this link: Thanks! |
Hi @jameshawkes, I followed the link, but it seems that it has not gone through fully. I got the following (attached). |
From: jameshawkes <[email protected]>
Sent: Friday, 16 August 2019 11:55 PM
To: ecmwf/eccodes <[email protected]>
Cc: Weiqing Qu <[email protected]>; Mention <[email protected]>
Subject: Re: [ecmwf/eccodes] Improvement on file spliting (#16)
Hi @wqu-bom<https://github.com/wqu-bom>, our bot was probably blocked by a firewall somewhere causing the error you received above -- although I could not replicate it. The bot is now on listening on port 80 so it should be accessible.
If you wouldn't mind, please try again at this link:
http:https://claassistant.ecmwf.int/ecmwf/eccodes?pullRequest=16
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#16?email_source=notifications&email_token=ABXJFLXWWKNIHL3G5WSRZNLQE2WSBA5CNFSM4IKFXMAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4OVSSI#issuecomment-522017097>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ABXJFLQGRVXA3FMBSUIZC6DQE2WSBANCNFSM4IKFXMAA>.
|
CLA signed successfully, thank you! |
This has now been merged to BitBucket and will appear in the next release of ecCodes. Many thanks |
Allow an input file to be split into each individual message (if nchunk=-1)