Skip to content

Only remove docker path prefix for local publication

Yoann Schneider requested to merge fix-models-path into master

When I tried to publish the kaldi models locally using my installed CLI, it failed because it tried to look for models in models/models/kaldi_ in stead of models/kaldi_.

This is because the current version expects to find a path like /usr/share/teklia/models/kaldi_ in .arkindex.yml but for kaldi (and maybe other workers as well), the path given is models/kaldi_ (whether this is the actual path or not is handled in the worker here). When given such a path, the current CLI version automatically adds a models/ prefix to the actual path when the teklia convention is not respected.

This is not a good behaviour imo. What would be better is to try both possibilities and return when one of those path exists.

Edited by Yoann Schneider

Merge request reports

Loading