move RenderConfig options from ChunkyOptions into separate class (long term goal is to reduce the amount of context which is pushed around for rendering with the RenderManager/RenderContext)

TODO later: split SceneIOProvider from RenderContext

Author: Maximilian Stiede

chunky/src/java/se/llbit/chunky/main/Chunky.java

@@ -217,7 +217,7 @@ public static void main(final String[] args) {
     if (cmdline.mode == CommandLineOptions.Mode.CLI_OPERATION) {
       exitCode = cmdline.exitCode;
     } else {
-      commonThreads = new ForkJoinPool(PersistentSettings.getNumThreads());
+      commonThreads = new ForkJoinPool(PersistentSettings.getRenderThreadCount());
 
       Chunky chunky = new Chunky(cmdline.options);
       chunky.headless = cmdline.mode == Mode.HEADLESS_RENDER || cmdline.mode == Mode.CREATE_SNAPSHOT;
@@ -341,7 +341,7 @@ public void update() {
    */
   public static ForkJoinPool getCommonThreads() {
     if (commonThreads == null) {
-      commonThreads = new ForkJoinPool(PersistentSettings.getNumThreads());
+      commonThreads = new ForkJoinPool(PersistentSettings.getRenderThreadCount());
     }
     return commonThreads;
   }

chunky/src/java/se/llbit/chunky/main/ChunkyOptions.java

@@ -17,12 +17,14 @@
 package se.llbit.chunky.main;
 
 import se.llbit.chunky.PersistentSettings;
-import se.llbit.chunky.renderer.RenderConstants;
+import se.llbit.chunky.renderer.RenderOptions;
+import se.llbit.chunky.renderer.ModifiableRenderOptions;
 import se.llbit.chunky.renderer.scene.Scene;
 
 import java.io.File;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.function.Consumer;
 
 /**
  * Various options for Chunky, set via the configuration file and/or command-line flags.
@@ -37,12 +39,10 @@ public class ChunkyOptions {
   public String imageOutputFile = "";
 
   private List<File> resourcePacks = new ArrayList<>();
-  public int renderThreads = -1;
   public File worldDir = null;
   public int target = -1;
 
-  public int tileWidth = RenderConstants.TILE_WIDTH_DEFAULT;
-  public int sppPerPass = RenderConstants.SPP_PER_PASS_DEFAULT;
+  protected ModifiableRenderOptions renderOptions = new ModifiableRenderOptions();
 
   /** Ignore scene loading errors when starting a headless render. */
   public boolean force = false;
@@ -56,7 +56,7 @@ private ChunkyOptions() {
   public static ChunkyOptions getDefaults() {
     ChunkyOptions defaults = new ChunkyOptions();
     defaults.sceneDir = PersistentSettings.getSceneDirectory();
-    defaults.renderThreads = PersistentSettings.getNumThreads();
+    defaults.renderOptions.setRenderThreadCount(PersistentSettings.getRenderThreadCount());
     return defaults;
   }
 
@@ -65,11 +65,19 @@ public static ChunkyOptions getDefaults() {
     clone.sceneDir = sceneDir;
     clone.sceneName = sceneName;
     clone.resourcePacks = new ArrayList<>(resourcePacks);
-    clone.renderThreads = renderThreads;
+    clone.renderOptions.copyState(renderOptions);
     clone.worldDir = worldDir;
     return clone;
   }
 
+  public RenderOptions getRenderOptions() {
+    return renderOptions;
+  }
+
+  public synchronized void changeRenderConfig(Consumer<ModifiableRenderOptions> callback) {
+    callback.accept(renderOptions);
+  }
+
   public List<File> getResourcePacks() {
     return resourcePacks;
   }

chunky/src/java/se/llbit/chunky/main/CommandLineOptions.java

@@ -294,13 +294,16 @@ public CommandLineOptions(String[] args) {
         arguments -> options.target = Math.max(1, Integer.parseInt(arguments.get(0))));
 
     registerOption("-threads", new Range(1),
-        arguments -> options.renderThreads = Math.max(1, Integer.parseInt(arguments.get(0))));
+        arguments -> options.changeRenderConfig(config ->
+          config.setRenderThreadCount(Math.max(1, Integer.parseInt(arguments.get(0))))));
 
     registerOption("-tile-width", new Range(1),
-        arguments -> options.tileWidth = Math.max(1, Integer.parseInt(arguments.get(0))));
+        arguments -> options.changeRenderConfig(config ->
+          config.setTileWidth(Math.max(1, Integer.parseInt(arguments.get(0))))));
 
     registerOption("-spp-per-pass", new Range(1),
-        arguments -> options.sppPerPass = Math.max(1, Integer.parseInt(arguments.get(0))));
+        arguments -> options.changeRenderConfig(config ->
+          config.setSppPerPass(Math.max(1, Integer.parseInt(arguments.get(0))))));
 
     registerOption("-version", new Range(0), arguments -> {
       mode = Mode.CLI_OPERATION;

chunky/src/java/se/llbit/chunky/map/WorldMapLoader.java

@@ -61,7 +61,9 @@ public WorldMapLoader(ChunkyFxController controller, MapView mapView) {
     mapView.addViewListener(this);
 
     // Start worker threads.
-    RegionParser[] regionParsers = new RegionParser[Integer.parseInt(System.getProperty("chunky.mapLoaderThreads", String.valueOf(PersistentSettings.getNumThreads())))];
+    RegionParser[] regionParsers = new RegionParser[Integer.parseInt(
+      System.getProperty("chunky.mapLoaderThreads", String.valueOf(PersistentSettings.getMapLoadingThreadCount()))
+    )];
     for (int i = 0; i < regionParsers.length; ++i) {
       regionParsers[i] = new RegionParser(this, regionQueue, mapView);
       regionParsers[i].start();

chunky/src/java/se/llbit/chunky/renderer/DefaultRenderManager.java

@@ -186,7 +186,9 @@ public DefaultRenderManager(RenderContext context, boolean headless) {
     this.bufferedScene = context.getChunky().getSceneFactory().newScene();
 
     // Create a new pool. Set the seed to the current time in milliseconds.
-    this.pool = context.renderPoolFactory.create(context.numRenderThreads(), System.currentTimeMillis());
+    this.pool = context.renderPoolFactory.create(
+      context.getRenderOptions().getRenderThreadCount(), System.currentTimeMillis()
+    );
     this.setCPULoad(PersistentSettings.getCPULoad());
 
     // Initialize callbacks here since java will complain `bufferedScene` is not initialized yet.
@@ -289,16 +291,16 @@ public void run() {
         getPreviewRenderer().sceneReset(this, reason, resetCount);
 
         // Select the correct renderer
-        Renderer render = mode == RenderMode.PREVIEW ? getPreviewRenderer() : getRenderer();
+        Renderer renderer = mode == RenderMode.PREVIEW ? getPreviewRenderer() : getRenderer();
 
         frameStart = System.currentTimeMillis();
         if (mode == RenderMode.PREVIEW) {
           // Bail early if the preview is not visible
           if (finalizeAllFrames) {
             // Preview with no CPU limit
             pool.setCpuLoad(100);
-            render.setPostRender(previewCallback);
-            render.render(this);
+            renderer.setPostRender(previewCallback);
+            startRender(renderer);
             pool.setCpuLoad(cpuLoad);
           }
         } else {
@@ -309,8 +311,8 @@ public void run() {
               updateRenderState(scene);
             });
           } else if (mode != RenderMode.PAUSED) {
-            render.setPostRender(renderCallback);
-            render.render(this);
+            renderer.setPostRender(renderCallback);
+            startRender(renderer);
           }
         }
 
@@ -323,6 +325,10 @@ public void run() {
     }
   }
 
+  protected void startRender(Renderer renderer) throws InterruptedException {
+    renderer.render(this);
+  }
+
   /**
    * Redraw the GUI screen. This should be run after postprocessing.
    */

chunky/src/java/se/llbit/chunky/renderer/ModifiableRenderOptions.java

@@ -0,0 +1,40 @@
+package se.llbit.chunky.renderer;
+
+public class ModifiableRenderOptions implements RenderOptions {
+  protected int renderThreadCount = -1;
+  protected int tileWidth = TILE_WIDTH_DEFAULT;
+  protected int sppPerPass = SPP_PER_PASS_DEFAULT;
+
+  public void copyState(RenderOptions other) {
+    renderThreadCount = other.getRenderThreadCount();
+    tileWidth = other.getTileWidth();
+    sppPerPass = other.getSppPerPass();
+  }
+
+  @Override
+  public int getRenderThreadCount() {
+    return renderThreadCount;
+  }
+
+  public void setRenderThreadCount(int renderThreadCount) {
+    this.renderThreadCount = renderThreadCount;
+  }
+
+  @Override
+  public int getTileWidth() {
+    return tileWidth;
+  }
+
+  public void setTileWidth(int tileWidth) {
+    this.tileWidth = tileWidth;
+  }
+
+  @Override
+  public int getSppPerPass() {
+    return sppPerPass;
+  }
+
+  public void setSppPerPass(int sppPerPass) {
+    this.sppPerPass = sppPerPass;
+  }
+}

chunky/src/java/se/llbit/chunky/renderer/PathTracingRenderer.java

@@ -60,7 +60,7 @@ public void render(DefaultRenderManager manager) throws InterruptedException {
     int cropX = scene.getCropX();
     int cropY = scene.getCropY();
 
-    int sppPerPass = manager.context.sppPerPass();
+    int sppPerPass = manager.context.getRenderOptions().getSppPerPass();
     Camera cam = scene.camera();
     double halfWidth = fullWidth / (2.0 * fullHeight);
     double invHeight = 1.0 / fullHeight;

chunky/src/java/se/llbit/chunky/renderer/RenderContext.java

@@ -64,11 +64,8 @@ public File getSceneDirectory() {
         return sceneDirectory;
     }
 
-    /**
-     * @return The preferred number of rendering threads.
-     */
-    public int numRenderThreads() {
-        return config.renderThreads;
+    public RenderOptions getRenderOptions() {
+      return config.getRenderOptions();
     }
 
     /**
@@ -84,18 +81,4 @@ public File getSceneFile(String fileName) {
         }
         return new File(sceneDirectory, fileName);
     }
-
-    /**
-     * @return The tile width.
-     */
-    public int tileWidth() {
-        return config.tileWidth;
-    }
-
-    /**
-     * @return The samples per pixel per pass
-     */
-    public int sppPerPass() {
-        return config.sppPerPass;
-    }
 }

chunky/src/java/se/llbit/chunky/renderer/TileBasedRenderer.java

@@ -65,7 +65,7 @@ public void setPostRender(BooleanSupplier callback) {
    *                 The second argument is the current pixel (x, y).
    */
   protected void submitTiles(DefaultRenderManager manager, BiConsumer<WorkerState, IntIntPair> perPixel) {
-    initTiles(manager);
+    initTiles(manager.bufferedScene, manager.context.getRenderOptions());
 
     cachedTiles.forEach(tile ->
         manager.pool.submit(worker -> {
@@ -86,11 +86,10 @@ protected void submitTiles(DefaultRenderManager manager, BiConsumer<WorkerState,
     );
   }
 
-  private void initTiles(DefaultRenderManager manager) {
-    Scene bufferedScene = manager.bufferedScene;
-    int width = bufferedScene.width;
-    int height = bufferedScene.height;
-    int tileWidth = manager.context.tileWidth();
+  private void initTiles(Scene scene, RenderOptions renderOptions) {
+    int width = scene.width;
+    int height = scene.height;
+    int tileWidth = renderOptions.getTileWidth();
 
     if (prevWidth != width || prevHeight != height) {
       prevWidth = width;
@@ -99,8 +98,10 @@ private void initTiles(DefaultRenderManager manager) {
 
       for (int i = 0; i < width; i += tileWidth) {
         for (int j = 0; j < height; j += tileWidth) {
-          cachedTiles.add(new RenderTile(i, FastMath.min(i + tileWidth, width),
-              j, FastMath.min(j + tileWidth, height)));
+          cachedTiles.add(new RenderTile(
+            i, FastMath.min(i + tileWidth, width),
+            j, FastMath.min(j + tileWidth, height)
+          ));
         }
       }
     }

chunky/src/java/se/llbit/chunky/ui/render/tabs/AdvancedTab.java

@@ -158,7 +158,7 @@ public PictureExportFormat fromString(String string) {
     renderThreads.setRange(1, 20);
     renderThreads.clampMin();
     renderThreads.onValueChange(value -> {
-      PersistentSettings.setNumRenderThreads(value);
+      PersistentSettings.setRenderThreadCount(value);
       renderControls.showPopup("This change takes effect after restarting Chunky.", renderThreads);
     });
 
@@ -281,7 +281,7 @@ public boolean shutdownAfterCompletedRender() {
   public void update(Scene scene) {
     outputMode.getSelectionModel().select(scene.getOutputMode());
     fastFog.setSelected(scene.fog.fastFog());
-    renderThreads.set(PersistentSettings.getNumThreads());
+    renderThreads.set(PersistentSettings.getRenderThreadCount());
     cpuLoad.set(PersistentSettings.getCPULoad());
     rayDepth.set(scene.getRayDepth());
     octreeImplementation.getSelectionModel().select(scene.getOctreeImplementation());