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

Allow updating binary header fields in SEG-Y factory. #131

Merged
merged 3 commits into from
Jun 19, 2024

Conversation

tasansal
Copy link
Contributor

@tasansal tasansal commented Jun 19, 2024

The create_binary_header function in factory now accepts an optional update parameter. This dictionary contains binary header fields that need to be modified from default. The function will update these fields with the values specified in the update dictionary before returning the encoded binary header.

Altay Sansal added 3 commits June 19, 2024 15:28
The create_binary_header function in factory.py now accepts an optional update parameter. This is a dictionary that contains binary header fields which need to be modified. The function will update these fields with the values specified in the update dictionary before returning the encoded binary header.
@tasansal tasansal added the enhancement New feature or request label Jun 19, 2024
@tasansal tasansal self-assigned this Jun 19, 2024
Copy link

codecov bot commented Jun 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.84%. Comparing base (2968eff) to head (f430f2b).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #131      +/-   ##
==========================================
+ Coverage   94.81%   94.84%   +0.03%     
==========================================
  Files          47       47              
  Lines        2062     2074      +12     
  Branches      245      247       +2     
==========================================
+ Hits         1955     1967      +12     
  Misses         81       81              
  Partials       26       26              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tasansal tasansal merged commit e0bca87 into main Jun 19, 2024
31 of 32 checks passed
@tasansal tasansal deleted the feature/factory-update-bin-hdr branch June 19, 2024 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant