Fix rendering missing data black and wrong segment id being displayed

frontend/javascripts/oxalis/model/bucket_data_handling/data_cube.js

@@ -515,12 +515,13 @@ class DataCube {
     // needs to be examined for rendering.
 
     const bucket = this.getBucket(this.positionToZoomedAddress(voxel, zoomStep));
+    const { renderMissingDataBlack } = Store.getState().datasetConfiguration;
 
     if (!(bucket instanceof DataBucket)) {
       // This is a NullBucket (e.g., because it's out of the bounding box or in a not-existing
       // magnification). This zoomstep is as good as all the other zoomsteps (as these will only
       // hold null buckets, too).
-      return true;
+      return renderMissingDataBlack;
     }
 
     if (bucket.hasData() || bucket.isLoaded()) {

frontend/javascripts/oxalis/model/bucket_data_handling/texture_bucket_manager.js

@@ -41,7 +41,7 @@ import type { ElementClass } from "types/api_flow_types";
 export const channelCountForLookupBuffer = 2;
 
 function getSomeValue<T>(set: Set<T>): T {
-  const value = set.values().next().value;
+  const { value } = set.values().next();
 
   if (value == null) {
     throw new Error("Cannot get value of set because it's empty.");
@@ -345,8 +345,8 @@ export default class TextureBucketManager {
         address = reservedAddress;
       }
 
-      const isBaseBucket = bucketZoomStep === currentZoomStep;
-      const isFirstFallback = bucketZoomStep - 1 === currentZoomStep;
+      const zoomStepDifference = bucketZoomStep - currentZoomStep;
+      const isBaseBucket = zoomStepDifference === 0;
       if (isBaseBucket) {
         if (address === -1) {
           let fallbackBucket = bucket.getFallbackBucket();
@@ -379,27 +379,22 @@ export default class TextureBucketManager {
           this.lookUpBuffer[posInBuffer] = address;
           this.lookUpBuffer[posInBuffer + 1] = bucketZoomStep;
         }
-      } else if (isFirstFallback) {
-        if (address !== -1) {
-          const baseBucketAddresses = this._getBaseBucketAddresses(bucket, 1);
-          for (const baseBucketAddress of baseBucketAddresses) {
-            const lookUpIdx = this._getBucketIndex(baseBucketAddress);
-            const posInBuffer = channelCountForLookupBuffer * lookUpIdx;
-            if (this.lookUpBuffer[posInBuffer] !== -2 || lookUpIdx === -1) {
-              // Either, another bucket was already placed here. Or, the lookUpIdx is
-              // invalid. Skip the entire loop
-              break;
-            }
-            this.lookUpBuffer[posInBuffer] = address;
-            this.lookUpBuffer[posInBuffer + 1] = bucketZoomStep;
+      } else if (address !== -1) {
+        const baseBucketAddresses = this._getBaseBucketAddresses(bucket, zoomStepDifference);
+        for (const baseBucketAddress of baseBucketAddresses) {
+          const lookUpIdx = this._getBucketIndex(baseBucketAddress);
+          const posInBuffer = channelCountForLookupBuffer * lookUpIdx;
+          if (this.lookUpBuffer[posInBuffer] !== -2 || lookUpIdx === -1) {
+            // Either, another bucket was already placed here. Or, the lookUpIdx is
+            // invalid. Skip the entire loop
+            break;
           }
-        } else {
-          // Don't overwrite the default -2 within the look up buffer for fallback buckets,
-          // since the effort is not worth it (only has an impact on the fallback color within the shader)
+          this.lookUpBuffer[posInBuffer] = address;
+          this.lookUpBuffer[posInBuffer + 1] = bucketZoomStep;
         }
       } else {
-        // Don't handle buckets with zoomStepDiff > 1, because filling the corresponding
-        // positions in the lookup buffer would take 8**zoomStepDiff iterations PER bucket.
+        // Don't overwrite the default -2 within the look up buffer for fallback buckets,
+        // since the effort is not worth it (only has an impact on the fallback color within the shader)
       }
     }
 

frontend/javascripts/oxalis/model/helpers/position_converter.js

@@ -175,11 +175,6 @@ export function getBaseBucketsForFallbackBucket(
     resolutions,
     betterZoomStep,
   );
-  if (zoomStepDifference > 1) {
-    // Due to the exponential complexity of calculating the "better bucket addresses",
-    // we currently only support this function if zoomStepDifference === 1
-    return [betterBucketAddress];
-  }
 
   // resolutionFactors is a [x, y, z] tuple with x, y, z being 1 or 2 each (because
   // zoomStepDifference === 1). In the case of isotropic resolutions, it's simply [2, 2, 2]

frontend/javascripts/oxalis/view/left-border-tabs/layer_settings_tab.js

@@ -608,24 +608,6 @@ class DatasetSettings extends React.PureComponent<DatasetSettingsProps, State> {
     Toast.success(`Successfully reloaded data of layer ${layerName}.`);
   };
 
-  onChangeRenderMissingDataBlack = async (value: boolean): Promise<void> => {
-    Toast.info(
-      value
-        ? messages["data.enabled_render_missing_data_black"]
-        : messages["data.disabled_render_missing_data_black"],
-      { timeout: 8000 },
-    );
-    this.props.onChange("renderMissingDataBlack", value);
-    const { layers } = this.props.datasetConfiguration;
-    const reloadAllLayersPromises = Object.keys(layers).map(async layerName => {
-      await clearCache(this.props.dataset, layerName);
-      await api.data.reloadBuckets(layerName);
-    });
-    await Promise.all(reloadAllLayersPromises);
-    window.needsRerender = true;
-    Toast.success("Successfully reloaded data of all layers.");
-  };
-
   getVolumeMagsToDownsample = (): Array<Vector3> => {
     if (this.props.task != null) {
       return [];