search

bgruening / galaxytools

:microscope::books: Galaxy Tool wrappers
MIT License
115 stars 222 forks source link

Update of label encoder to allow storing of the model #1324

Open paulzierep opened 11 months ago

paulzierep commented 11 months ago

Something was messed with my rebase. New clean PR. This update enables the export of the label encoder as h5 model, that can be used to encode new data as well, similarly to the use of pretrained models.

paulzierep commented 11 months ago

@bgruening @anuprulez how should the tool version be bumped here ? These tools do not follow the @TOOL_VERSION@+galaxy@SUFFIX_VERSION@ logic, they all use the @VERSION@ tag set in the main_macros.xml ?

bgruening commented 11 months ago

Hardcode it without macro. We need to shift to IUC standards at some point.

anuprulez commented 11 months ago

@paulzierep I followed version="@VERSION@" profile="@PROFILE@" pattern while upgrading ML tools to 0.10.0. But, it should not make much difference at the moment if they will be changed according to IUC standards :)

paulzierep commented 11 months ago

Then I would propose to use @VERSION@ from the main_macros.xml and add +galaxy@VERSION_SUFFIX@ to the tools individually

paulzierep commented 11 months ago

added the VERSION_SUFFIX can this be merged @anuprulez @bgruening

anuprulez commented 10 months ago

I think there are a few linting errors that need to be fixed