malloc new_gather, fail fast false in CI, return pointer to gather st…

…ruct

.github/workflows/cmake.yml → .github/workflows/CI.yml

@@ -2,9 +2,9 @@ name: Clang CI
 
 on:
   push:
-    branches: ["*"]
+    branches: [ "*" ]
   pull_request:
-    branches: ["main"]
+    branches: [ "main" ]
 
 env:
   # Customize the CMake build type here (Release, Debug, RelWithDebInfo, etc.)
@@ -13,8 +13,9 @@ env:
 jobs:
   build_test:
     strategy:
+      fail-fast: false
       matrix:
-        os: [ubuntu-latest, macos-latest]
+        os: [ ubuntu-latest, macos-latest ]
     # The CMake configure and build commands are platform-agnostic and should work equally well on Windows or Mac.
     # You can convert this to a matrix build if you need cross-platform coverage.
     # See: https://docs.github.com/en/free-pro-team@latest/actions/learn-github-actions/managing-complex-workflows#using-a-build-matrix

src/libseis.c

@@ -45,23 +45,20 @@ float *gain(const float *data, int nt, int nx, float dt, float pow) {
     return gained_data;
 }
 
-int gain_cmp(Gather *gather, float pow) {
-    Gather new_gather = {
-        gather->id,
-        gather->nt,
-        gather->nx,
-        gather->dt,
-        malloc(gather->nt * gather->nx * sizeof(float)),
-
-    };
-    float *new_data = malloc(gather->nt * gather->nx * sizeof(float));
+Gather *gain_gather(Gather *gather, float pow) {
+    Gather *new_gather = malloc(sizeof(struct Gather));
+    new_gather->id = gather->id;
+    new_gather->nt = gather->nt;
+    new_gather->nx = gather->nx;
+    new_gather->dt = gather->dt;
+    new_gather->data = malloc(gather->nt * gather->nt * sizeof(float));
     for (int trace_index = 0; trace_index < gather->nx; trace_index++) {
         for (int sample_index = 0; sample_index < gather->nt; sample_index++) {
-            float time = (float) sample_index * new_gather.dt;
+            float time = (float) sample_index * new_gather->dt;
             float t_pow = powf(time, pow);
-            int data_index = (trace_index * new_gather.nt) + sample_index;
-            new_gather.data[data_index] = gather->data[data_index] * t_pow;
+            int data_index = (trace_index * new_gather->nt) + sample_index;
+            new_gather->data[data_index] = gather->data[data_index] * t_pow;
         }
     }
-    return 0;
+    return new_gather;
 }

src/libseis.h

@@ -40,7 +40,7 @@ float *read_float(char path[], int nt, int nx);
 void write_float(char path[], float *data, int nt, int nx);
 
 /**
- * gain_cmp
+ * gain_gather
  *
  * @details
  * Applies a time-variant scaling to a series of seismic traces in the same gather.
@@ -58,4 +58,4 @@ void write_float(char path[], float *data, int nt, int nx);
  * All gather attributes, besides the seismic data, are copied from the original gather
  *
  */
-int gain_cmp(Gather *gather, float pow);
+Gather *gain_gather(Gather *gather, float pow);

test/libseis_test.cc

@@ -67,6 +67,7 @@ TEST(LibseisIO, ReadWriteFloat) {
 
 TEST(GainCmp, GainsCmpGather) {
     float t_pow = 2;
-    int foo = gain_cmp(&cmp, t_pow);
-    EXPECT_EQ(0, foo);
+    Gather *foo = gain_gather(&cmp, t_pow);
+    printf("%d\n", foo->nt);
+    EXPECT_EQ(0, 0);
 }