marwan-abdellah
commented
11 months ago Dear @faberno Thanks for your kind help. To be able to add you in the contributors list, would you please make a pull request so I can merge it to have your contribution shown in the repo.
Hey, first of all thanks for this great library, it's a big help. I wanted to report some bugs I encountered when using the CLI to reconstruct a mesh.
The tessellation-level option is called tessellation-ratio in the code. This fails when _arguments.tessellationratio is called https://github.com/BlueBrain/VessMorphoVis/blob/90446383cef5caa4f9a8d8613a4a9a04c293fdf6/vmv/options/vessmorphovis_options.py#L171 Quickfixing this by changing the line to _arguments.tessellationlevel allows the program to continue, but setting the tessellation to another value than 1.0 will cause a _TypeError: Converting py args to operator properties: : keyword "applyas" unrecognized in the mesh builder.
vmv.file does not have a function _read_morphology_fromfile https://github.com/BlueBrain/VessMorphoVis/blob/90446383cef5caa4f9a8d8613a4a9a04c293fdf6/vmv/interface/cli/mesh_reconstruction.py#L298
_only cli_options.mesh.exportobj is used to determine if _vmv.file.export_meshobject should be called, this ignores for example export-vascular-mesh-stl https://github.com/BlueBrain/VessMorphoVis/blob/90446383cef5caa4f9a8d8613a4a9a04c293fdf6/vmv/interface/cli/mesh_reconstruction.py#L112
_vmv.file.export_meshobject gets wrong arguments https://github.com/BlueBrain/VessMorphoVis/blob/90446383cef5caa4f9a8d8613a4a9a04c293fdf6/vmv/interface/cli/mesh_reconstruction.py#L113-L116
Even when _meshobjects from _vmv.scene.get_list_of_meshes_inscene() is just one mesh, it's still a list object https://github.com/BlueBrain/VessMorphoVis/blob/90446383cef5caa4f9a8d8613a4a9a04c293fdf6/vmv/interface/cli/mesh_reconstruction.py#L100-L109 I just fixed it by setting mesh_object = mesh_objects[0] when len(mesh_objects) == 1
When these points were fixed, everything worked as expected. If you have any questions I can provide more accurate descriptions.