Yarroudh / Optim3D

Command-Line Interface (CLI) application for efficient and scalable generation of large-scale 3D building models.

Home Page:https://optim3d.readthedocs.io/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Rename reconstruct_.json for each tile

Mboga opened this issue · comments

commented

Thanks for the tool.

I am wondering in line 453 of reconstruct.json, if you should rename the reconstruct_.json for each of the new tile. This would happen after line 499 in main.py as data['nodes']['NestedFlowchart']['parameters'] = 'reconstruct{}_'.format(i) ?

Thank you

Thank you for feedback @Mboga
I will look at your proposition soon to see what can be adjsuted or done.

Apparently, when changing the filepath to reconstruct{i}_.json, the reconstruction fails completely as it cannot point to the right file.

These are the logs of the reconstruct file after adding:
data['nodes']['NestedFlowchart']['parameters'] = 'reconstruct{}_'.format(i)

logs.txt

@Mboga can you try with this on line 499 instead of what you proposed:

data['nodes']['NestedFlowchart']['parameters']['filepath'] = '{}/flowcharts/reconstruct{}_.json'.format(output, i)

commented

Thanks @Yarroudh, this solves the error in the log.txt but only lod0 reconstruction takes place when I use the exact workflow. I will recheck to ensure that all is correct in the new reconstruct.json and reconstruct_.json files

commented

@Yarroudh, re-checked to ensure all paths are correct- and subsequently achieved successful lod22 reconstruction :)

Great. Thanks for your feedback.