cannot open catproduct/SE assemblies if folder same location

[syscordan]

🔍 Before submitting the issue

• I have searched among the existing issues
• I am using a Python virtual environment

🐞 Description of the bug

upload_file is scanning the dir and uploading all of the individual files. It looks like it wasn't coded to handle dirs.

Here's the failure point
pyansys-geometry/src/ansys/geometry/core/modeler.py at main · ansys/pyansys-geometry

The loop just needs to skip dirs

📝 Steps to reproduce

Ensure the location the CATProduct is at has no folder in it
willnotopen = Path("C:\Assembly\Jet_Engine_Assembly", "Exhaust_Assembly.CATProduct")
design = modeler.open_file(willnotopen)
The model can open.
Now create a folder in the same location as the CATProduct and run the above code. You get an error about it wanting a file and not a directory.

Jet_Engine_Assembly.zip

💻 Which operating system are you using?

Windows

📀 Which ANSYS version are you using?

261

🐍 Which Python version are you using?

3.10

📦 Installed packages

import os
import sys
import math
import numpy as np
from pint import Quantity
from pyvista.plotting import system_supports_plotting
from ansys.geometry.core import *
from ansys.geometry.core.connection import *
from ansys.geometry.core.designer import *
from ansys.geometry.core.materials import *
from ansys.geometry.core.math import *
from ansys.geometry.core.misc import *
from ansys.geometry.core.misc.units import UNITS
from ansys.geometry.core.plotting import *
from ansys.geometry.core.sketch import *
from ansys.geometry.core.designer.part import *
from ansys.geometry.core.designer.part import *
from ansys.geometry.core.misc import DEFAULT_UNITS, UNITS, Accuracy, Angle, Distance, checks
import pytest
from pathlib import Path

[RobPasMue]

This one should be an easy fix but... I'm curious. I think @jonahrb implemented this part and he was expecting always a flat directory for CATIA projects. Is this no longer true? We should verify this before moving forward

[RobPasMue]

@jonahrb please confirm whenever you can

[jonahrb]

@RobPasMue @syscordan i suppose i implemented a flat directory just because that's how all the examples i had were, if catia assemblies can have nested directories we should absolutely support that

[jonahrb]

@RobPasMue @syscordan so i guess the next question is should we just skip over directories or recurse into them and upload all the files in there?

[RobPasMue]

If there can be nested directories then... We need to recurse into all of them

[syscordan]

@RobPasMue @jonahrb you should skip over them, but let me check other assembly files to see ensure that none of them have extra files that are nested in the folder. When I tested this out, my folders were empty or contained an extra copy of a file. This also happens for solidege assemblies too, but not solidworks assemblies

[RobPasMue]

@RobPasMue @jonahrb you should skip over them

Oh so you are saying that the nested directories are not needed?

[syscordan]

@RobPasMue you are correct. I am double checking to ensure that I find no assemblies that need the nested folders

[syscordan]

@RobPasMue @jonahrb, so far all I have found where an assembly uses a nest folder for parts is JT which I testing now to see if I can open and plot the bodies

[syscordan]

I found a Solidedge assembly that had files within it nested folders which were needed

[RobPasMue]

Okay - so we sync the entire directory, including nested directories