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

Updating Web-Server Models for Live Release #74 #12

Closed
SimonDaKappa opened this issue Sep 9, 2024 · 1 comment · Fixed by #15
Closed

Updating Web-Server Models for Live Release #74 #12

SimonDaKappa opened this issue Sep 9, 2024 · 1 comment · Fixed by #15
Labels
enhancement New feature or request From VidToNerf

Comments

@SimonDaKappa
Copy link
Collaborator

From NeRF-or-Nothing/vidtonerf#74

@PotatoPalooza
Member
PotatoPalooza commented on Mar 1, 2023
Currently the web-server's model's are outdated and do not properly encapsulate the features we are planning to add for the MVP live release. The current DB scheme will have to be rewritten to further separate currently nested objects and allow greater flexibility with adding new queries and features. These new schema should be documented in the refactor section of the web-server readme.

Additionally the new models should centralize file operations. All loading and saving of image and video data should be handled in the models section of the code so the data base and file operations are hidden from the rest of the code base. We will need to decide if Video and Image objects are able to save themselves or if additional managers should handle these operations.

I recommend checking out this video as it goes into a design pattern perfect for this issue. Once these models are complete we will begin updating the rest of the codebase to use this new scheme.

@SimonDaKappa
Copy link
Collaborator Author

Should be handled by V3. ALmost all services are now injected and further decoupled. DB Schemas have been expanded for more functionality. See go-web-server wiki for DB changes

@SimonDaKappa SimonDaKappa added enhancement New feature or request From VidToNerf labels Sep 9, 2024
@SimonDaKappa SimonDaKappa linked a pull request Sep 9, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request From VidToNerf
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant