Skip to content
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

Refactor to factory paradigm #21

Closed
alxdncn opened this issue Jun 29, 2021 · 2 comments
Closed

Refactor to factory paradigm #21

alxdncn opened this issue Jun 29, 2021 · 2 comments
Assignees
Labels
enhancement Request an improvement of an existing feature

Comments

@alxdncn
Copy link
Collaborator

alxdncn commented Jun 29, 2021

This is a refactor to improve stability and scalability of new features.

@alxdncn alxdncn added the enhancement Request an improvement of an existing feature label Jun 29, 2021
@alxdncn alxdncn added this to the AliceLG version 1.2 milestone Jun 29, 2021
@alxdncn alxdncn added this to In progress in Blender Add-On Unification Jun 29, 2021
@regcs
Copy link
Owner

regcs commented Jul 13, 2021

Status of the Refactor

General

  • Add a log file mechanism
  • Shift the add-on preferences to a separate file (preferences.py)
  • Implement pyLightIO as an external package to handle the lightfield display communication
  • Shift all UI related code to a separate file (ui.py)
  • Handle installation of required python packages
  • Include pyLightIO in the bundle (this will be changed in later versions to a pipinstall)
  • Final code clean-up

Lightfield Viewport / Live View

  • Remove freeHPC & lonetech's lightfield shaders from the code base
  • Clean-up the codebase of the Lightfield Viewport
  • Use HoPS via pyLightIO for the Lightfield Viewport: The separate Blender window is gone.
  • Re-implement the quilt viewer
  • Remove holographic / lightfield cursor (see Product definition for lightfield / holographic cursor #49)
  • Re-implement multi-scene handling (currently the lightfield viewport shuts down on scene changes / creation)
  • Handle lightfield viewport behavior when no camera is selected: Request displaying the LG demo quilt from HoPS

Quilt Renderer

  • Remove redundant code parts for file naming
  • Optimize operator logic for flexibility and to enable additional implementation of the multiview camera systems

@regcs regcs moved this from In progress to Done in Blender Add-On Unification Jul 27, 2021
@regcs
Copy link
Owner

regcs commented Jul 27, 2021

Everything I wanted to refactor for version 2.0 should be done now. I am sure, there are other areas that still can be improved in the code base, but I leave those for the future.

@regcs regcs closed this as completed Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Request an improvement of an existing feature
Projects
No open projects
Development

No branches or pull requests

2 participants