Feature/custom model editor#426
Feature/custom model editor#426SciLor merged 10 commits intotoniebox-reverse-engineering:developfrom
Conversation
|
Please use the web repo for the web stuff. |
4a44e68 to
0127d1e
Compare
Sure .. Second PR will be created for submodule changes |
…anagement - Backend: tonies.custom.json save API with validation and backups - Backend: custom_img root support for image workflows - Frontend: Full editor for custom models (add, edit, delete) - Frontend: Image picker and file browser integration - Remove built web assets (now built from teddycloud_web submodule)
0127d1e to
95b2482
Compare
|
Please also remove the compiled web changes (contrib www) |
|
This would solve the following issues: |
- Remove custom_editor_preview setting (editor always active) - Ignore contrib/data/www/ (compiled web assets) - Update teddycloud_web submodule (v0.7.0, PR feedback) Made-with: Cursor
removed and added to .gitignore. |
c738c7d to
68eef21
Compare
- Added ca-certificates to the list of packages installed in the build environment.
68eef21 to
d796621
Compare
46294f0 to
14465ee
Compare
There was a problem hiding this comment.
This helped me really much ti develop new features. On the other hand I am not happy about adding binaries to the repo although they are small.
I f not wented ti can werork my history to remove al binary blobs again. However for useabilety during development it might be worth to keep this.
Also this couldbe used to start up a demo instance users can check out?!
There was a problem hiding this comment.
This helped me really much ti develop new features. On the other hand I am not happy about adding binaries to the repo although they are small.
You mean the TAFs and the images? You can keep them in the repo, if those are free to use. The images look AI-generated. Are the TAFs, too?
Also this couldbe used to start up a demo instance users can check out?!
I don't think this makes sense. This will end up in a mess. People will start to attack this instance, fill it with copyright material etc.
There was a problem hiding this comment.
Images are AI, created by me. i assume they are therefore free to use? I at least hereby confirm that everybody can use them ans Attribution-NonCommercial-ShareAlike.
Taf: i struggled to find a really short audio. I ended up using the ICQ message tone that is free to download on a vast amount of pages. All taf are that audio (but wit modified header to have different ID and hashes) -- i guess you have a point there. This probably is a copyright issue.
Ill rework, remove from history and try to find alternative.
There was a problem hiding this comment.
There was a problem hiding this comment.
already had found https://pixabay.com/sound-effects/musical-comedy-stinger-quick-joke-cue-469097/).
that is now used. sadly somehow the header replacement did not work as expected so i need to investigate my custom models in dev-sandbox tomorrow
|
Please reset the deletions you did in "contrib/data/www/". This means, use the files in the current develop. Also please reset the submodule teddycloud_web to the one in develop. |
681e292 to
689ce65
Compare
- Introduced a new `dev-sandbox` directory for running TeddyCloud with mock data. - Added Docker support for local development, including setup and teardown scripts. - Included configuration files and sample data for testing. - Updated `.gitignore` to exclude sandbox runtime data. - Enhanced README with instructions for using the development sandbox. - Use free audio content for audio_*.taf (source: https://pixabay.com/sound-effects/musical-comedy-stinger-quick-joke-cue-469097/).
- Added functions to resolve cached URLs to original URLs when cache is disabled. - Implemented cache entry creation for redirects based on index lookup. - Updated cache handling in the server to utilize the persistent index for missing entries. - Enhanced JSON handling to persist original URLs instead of cached paths for image loading.
…oad timeout - Restore contrib/data/www/ from develop - Reset teddycloud_web submodule to develop - Add core.file_upload_timeout_ms
- Replaced the old `docker-compose.test.yaml` with a new `docker-compose.yaml` for simplified Docker setup. - Introduced a new `Dockerfile` for building the development environment with necessary dependencies. - Updated `Makefile` to reflect changes in Docker commands and improve output formatting. - Modified `setup.sh` to ensure proper directory structure and file copying for the sandbox. - Updated `README.md` with clearer usage instructions for the development sandbox. - Adjusted JSON configuration files for tonies to ensure correct audio source references. - Added new content JSON files for additional tonies models.
689ce65 to
039c5a9
Compare
|
Could you give me a brief description, what the caching changes do exactly and which problem you solved? |
From now on we Store mapping from Cached image path to original URL (tab separatd file). When cache is enabled the backend uses that to serve the correct file from cache. When saving a model we always store the url, that way if you lose your cache, the images can't still be ob change (and the cache can be recreated). In case production environment misses to back up the cache dir (which I did 🤦 ) you should be able to recover. |
Shouldn’t be the lost images be automatically fetched and cached again? Is that not already implemented? |
|
Please split the cache changes from the custom model editor changes, as this currently blocks the merge. |
|
After looking into this after my vaccation more carefully, I realize my earlier answer was inaccurate -- I had misunderstood how the existing cache works. The cache is already robust by design: it uses a deterministic SHA256 hash of the original URL as the filename, and the JSON files ( My original issue was caused by my web logic for custom models. Here I did not always store the original URL in the custom.json but sometimes the image selector returned the My cache changes (persistent index, fallback resolution) therefore were essentially an unnecessary safety net based on a wrong assumption that |
57b436a
into
toniebox-reverse-engineering:develop
|
The build fails, maybe some change in the makefile is problematic |
I will have a look at this later -- on my machine it worked in the Devcontainer 😜 |
This Adds
I also removed built web assets from teddycloud_web submodule -- i assume that was added by accident?!)REQUIRED BY: Web changes from toniebox-reverse-engineering/teddycloud_web#283
Dev Sandbox
Added Dev sandbox that can be used locally to check changes made in the web ui. The Sandbox adds several custom tonies, models, images ad audio files
Preview -- what will these backend changes make available in web
Custom Tonies Editor: Editor for managing custom tonie models
Settings: Enable/Disable Editor -- (changes menu entry from

WIP: Add ModeltoManage custom modelsCustom tonies editor

Manage custom modelsImage Manager:
Shared UI for selecting and managing images in TeddyStudio and Custom Tonies Editor
Image Editor::Default Images

Image Editor::Upload and manage Custom Images -- also available in Tonie Studio
