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

Improvement on file spliting #16

Merged
merged 3 commits into from
Aug 22, 2019
Merged

Improvement on file spliting #16

merged 3 commits into from
Aug 22, 2019

Conversation

wqu-bom
Copy link
Contributor

@wqu-bom wqu-bom commented Aug 8, 2019

Allow an input file to be split into each individual message (if nchunk=-1)

Allow an input file to be split into each individual message (if nchunk=-1)
@FussyDuck
Copy link

FussyDuck commented Aug 8, 2019

CLA assistant check
All committers have signed the CLA.

@shahramn
Copy link
Collaborator

shahramn commented Aug 8, 2019

Thank you. Have you included tests?
Please see: tests/codes_split_file.sh

Added the third test for nchunk=-1. OUTPUT_FILENAME_FORMAT changed to 3 digits, which led to file name changes in this script.
@shahramn
Copy link
Collaborator

shahramn commented Aug 9, 2019

Thank you for the test. By the way have you signed the Contributor License Agreement?

@dtip
Copy link
Contributor

dtip commented Aug 9, 2019

Hi chaps. Can I suggest you switch the osx_image in .travis.yml to xcode10.2? That may fix the MacOS build issue.

@shahramn
Copy link
Collaborator

shahramn commented Aug 9, 2019

Thanks Daniel. I will change it and push to develop

@wqu-bom
Copy link
Contributor Author

wqu-bom commented Aug 11, 2019

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.

@StephanSiemen
Copy link
Contributor

StephanSiemen commented Aug 12, 2019

Hi @wqu-bom,
I signed the code agreement fine (see comment above). The system uses email addresses to identify users and contributions. The system relies on you using the same email address for the GIT checkins and the GitHub account. Could you please check you use the same email?

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
Thanks

@wqu-bom
Copy link
Contributor Author

wqu-bom commented Aug 13, 2019

Tried the "CLA not signed yet" button, but it did not work (error msg below), nor the link "Details" above.
Description: internal error - server connection terminated

@jameshawkes
Copy link

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:
http:https://claassistant.ecmwf.int/ecmwf/eccodes?pullRequest=16

Thanks!

@wqu-bom
Copy link
Contributor Author

wqu-bom commented Aug 19, 2019

Hi @jameshawkes, I followed the link, but it seems that it has not gone through fully. I got the following (attached).

@wqu-bom
Copy link
Contributor Author

wqu-bom commented Aug 19, 2019 via email

@jameshawkes
Copy link

jameshawkes commented Aug 19, 2019

Hi @wqu-bom, instead of an attachment I got a copy of the email notification on Github. Can you upload it directly on the Github PR instead or via email?

(You can attach files in the reply box at the bottom)
#16

Otherwise just send me the attachment at

Thanks for your patience.

@jameshawkes
Copy link

CLA signed successfully, thank you!

@shahramn shahramn merged commit 89a8a53 into ecmwf:develop Aug 22, 2019
@shahramn
Copy link
Collaborator

This has now been merged to BitBucket and will appear in the next release of ecCodes. Many thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants