Skip to content

New Extension Spec Compatibility and Usability Improvements #36

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

Merged

Conversation

digitalillusions
Copy link
Member

This pull request includes multiple changes to improve the structure and functionality of the Blender Sequence Loader add-on. The most important changes include the creation of a new manifest file, updates to import statements, and enhancements to the execute method in the operators.py file.

New Extension Spec Compatibility:

  • Added a new blender_manifest.toml file to define the add-on's metadata, such as ID, version, name, maintainer, supported Blender versions, license, and bundled third-party Python modules.

Code Structure Improvements:

  • Updated import statements to use relative imports, improving module encapsulation and reducing potential import errors. (bseq/__init__.py, bseq/importer.py) [1] [2]
  • Changed the bl_idname in BSEQ_addon_preferences to use __package__ for better compatibility and maintainability. (bseq/operators.py)

Functionality Enhancements:

  • Enhanced the execute method in operators.py to allow for recursive loading of sequences in nested folder structures, but handling nested collections more robustly, ensuring proper linking and unlinking of collections and freeing unused datablocks. (bseq/operators.py) [1] [2] [3]

Code Cleanup:

  • Commented out unnecessary import statements and adjusted the way the bseq module is referenced in the subscribe_to_selected and unsubscribe_to_selected functions to avoid potential issues. (bseq/messenger.py) [1] [2]

Additional Changes:

  • Updated the download_wheels.sh script to include specific versions of third-party Python modules required by the add-on. (download_wheels.sh)

by default it will now detect common prefixes and eliminate them. in the future it might be interesting to have an option to keep prefixes and simply use blenders naming to create a new one with an incremented name.
@digitalillusions digitalillusions merged commit 46044aa into InteractiveComputerGraphics:main Nov 20, 2024
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