search

sixty-north / segpy

A Python package for reading and writing SEG Y files.
Other
102 stars 54 forks source link

Replaced a call to HeaderPacker by make_header_packer #20

Closed rth closed 9 years ago

rth commented 9 years ago

Fixes an issue in segpy/writer.py that was probably not fully adapted to the last change in the API. In particular, with the last version of the module, running, example/loadsave.py raises an exception since HeaderPacker is called with 2 arguments while it expects 3.

This PR fixes that problem.

rob-smallshire commented 9 years ago

Thank you for spending time investigating this issue.

To merge your pull request I would need you to agree to the contributors license agreement. Instructions on how to do this can be found in CONTRIBUTING.rst. This is so we can keep the ownership of the Segpy code simple. If you're unable to do so, please let me know.

rth commented 9 years ago

Thanks for your quick response.

Sure, I don't mind signing the contributors license agreement. The only thing is that both links to the pdf in CONTRIBUTING.rst are broken, so I can't download the agreement.

rob-smallshire commented 9 years ago

Sorry about that – the links are fixed now.