diff --git a/wagl/ancillary.py b/wagl/ancillary.py
index 2e3e49e5..1ad599db 100644
--- a/wagl/ancillary.py
+++ b/wagl/ancillary.py
@@ -559,7 +559,7 @@ def collect_nbar_ancillary(
         for acq in container.get_acquisitions(group=group):
             if acq.band_type is not BandType.REFLECTIVE:
                 continue
-            data = get_brdf_data(acq, brdf_dict, compression)
+            data = get_brdf_data(acq, brdf_dict, compression=compression)
 
             # output
             for param in data:
diff --git a/wagl/brdf.py b/wagl/brdf.py
index 942d2986..62d228e3 100644
--- a/wagl/brdf.py
+++ b/wagl/brdf.py
@@ -555,9 +555,9 @@ def assert_exists(path):
 def get_brdf_data(
     acquisition,
     brdf_config,
-    mode=None,
     compression=H5CompressionFilter.LZF,
     filter_opts=None,
+    mode=None,
 ):
     """
     Calculates the mean BRDF value for the given acquisition,