search

scheckmedia / photils-cli

photils-cli is an application that passes an image through a neural network, classifies it, and extracts the suggested tags. Everything happens offline without the need that your data are sent over the internet.
GNU General Public License v3.0
42 stars 5 forks source link

Release does not support label override #7

Closed gnmackenzie closed 2 years ago

gnmackenzie commented 2 years ago

Thank you very much for this utility.

The linux appimage release does not include the label override feature. I compiled a new appimage from the latest source and the feature worked.

Thanks again.

mpaglia0 commented 2 years ago

Same problem here. I am testing the Italian translation but labels override cannot work. Started <darktable -d lua> and see following error messages:

7,402515 LUA use export to for prediction [_dev_auto_apply_presets] missing mandatory module rawprepare for image 11201 [_dev_auto_apply_presets] missing mandatory module temperature for image 11201 [_dev_auto_apply_presets] missing mandatory module highlights for image 11201 [_dev_auto_apply_presets] missing mandatory module demosaic for image 11201 [_dev_auto_apply_presets] missing mandatory module colorin for image 11201 [_dev_auto_apply_presets] missing mandatory module colorout for image 11201 [_dev_auto_apply_presets] missing mandatory module gamma for image 11201

scheckmedia commented 2 years ago

Yes the last commit was not able to run on all CI stages and therefore the release was not published. I'll have a look to migrate everything to Githubs' actions to solve this issue. Alternatively, you can build everything from the source as @gnmackenzie mentioned.

scheckmedia commented 2 years ago

@mpaglia0 Seems I make some progress with the refactoring and GitHub actions. Can you check whether you can download the artifacts here: https://github.com/scheckmedia/photils-cli/suites/7197260554/artifacts/287704763

This will be a very early beta which also replaces some unnecessary dependencies.

mpaglia0 commented 2 years ago

@scheckmedia works good! Now I can see labels in Italian. Moreover my terminal shows no messages with <darktable -d lua> Thanks indeed!! Maurizio

scheckmedia commented 2 years ago

Molto bene! 😄 I'll prepare the release as soon as possible. Thanks for testing.

scheckmedia commented 2 years ago

Fixed by #9

gnmackenzie commented 2 years ago

Thank you for your work it is most appreciated.