Closed swheaton closed 2 months ago
The recent updates focus on enhancing asset path management within the Scene3D
class and the data exporters of a software library. A new method, update_asset_paths
, centralizes the process of updating paths based on a mapping dictionary. This method is now utilized in the data exporters to streamline path updates. Additionally, new unit tests ensure the functionality behaves as expected across various scenarios.
File Path | Change Summary |
---|---|
.../core/threed/scene_3d.py |
Added update_asset_paths method to Scene3D class for updating asset paths. |
.../utils/data/exporters.py |
Enhanced asset path handling using the new update_asset_paths from Scene3D . |
.../unittests/.../import_export_tests.py |
New test class ThreeDMediaTests with methods to test asset path updates in various scenarios. |
🐰✨ In the code where paths intertwine,
A rabbit hops with updates fine.
Paths now clear where once were knots,
Through tests and maps, it plots its spots.
Celebrate the code, fresh and new,
Hop along, there's more to view!
🌟🐇
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
make a batch media exporter that does this in bulk instead of one at a time. Then re-use that for fos.upload_media() and download_media() in Teams so we don't have the same fragile code implemented twice. With the side benefit that export should be faster.
Ok I am seeing that this exists, sort of, in Teams when a cloud path is involved. I am unable to take this refactor on right now so perhaps I'll have to leave it alone for now.
@coderabbitai review
Checkpointing my work in case I don't get to finish it.
rel_dir
) causes a file to be renamed from, say,pcd.pcd
topcd-2.pcd
The tests are absolutely horrendous on the eyes, I'm so sorry!! They could be refactored to something nicer but didn't have time right now.
What I would like to do if I have time:
asset_dir
fos.resolve()
and a cloud friendlyos.path.relpath()
if it doesn't already exist. Actually the latter probably just works but we would want to double check.fos.upload_media()
anddownload_media()
in Teams so we don't have the same fragile code implemented twice. With the side benefit that export should be faster.Summary by CodeRabbit