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

addition of multi-render app support and log output adjustments #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

av-romann
Copy link

(not sure what's gone wrong with the spacing. I may withdraw and resubmit if it's an issue)

The configuration and rendering has been updated to allow for an arbitrary selection of multiple render tools, with independent render control.
There's some small pre-work done to allow selective triggering of render apps via the console, but that part hasn't been implemented yet.
There are also some console output changes to better distinguish which component is doing the reporting.
This is a breaking change to the config file format, which now looks like this (first version, without rationalisation of properties):

 "Renders": {
    "Global": {
      "EnableRenders": true,
      "RenderAppBinPath": "",
      "RenderAppOutputPath": "",
      "RenderInterval": 240.0,
      "RenderAppGlobalArgs": "",
      "RenderAppTasks": [],
      "LowPriority": false
    },
    "bedrock-viz": {
      "EnableRenders": true,
      "RenderAppBinPath": "C:\\Minecraft\\Tools\\bedrock-viz.1.7\\bedrock-viz.exe",
      "RenderAppOutputPath": "test\\bedrock-viz",
      "RenderInterval": 240.0,
      "RenderAppGlobalArgs": " --shortrun --no-force-geojson --detail --db $WORLD_PATH --outdir $OUTPUT_PATH ",
      "RenderAppTasks": [
        "--html-most --slices --all-image"
      ],
      "LowPriority": false
    },
    "papyruscs": {
      "EnableRenders": true,
      "RenderAppBinPath": "C:\\Minecraft\\Tools\\papyruscs\\PapyrusCs.exe",
      "RenderAppOutputPath": "test\\papyruscs",
      "RenderInterval": 240.0,
      "RenderAppGlobalArgs": "-w $WORLD_PATH -o $OUTPUT_PATH --htmlfile index.html -f webp -q -1 --rendermode Heightmap",
      "RenderAppTasks": [
        "--dim 0",
        "--dim 1",
        "--dim 2",
        "--dim 0 -p underground",
        "--dim 0 -p aquatic"
      ],
      "LowPriority": false
    }

the configuration and rendering has been updated to allow for an arbitrary selection of multiple render tools, with independent render control.
there's some small pre-work done to allow selective triggering of render apps via the console, but that part hasn't been implemented yet.
there are also some console output changes to better distinguish which component is doing the reporting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant