Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Disambiguate passing multiple datasets to apply #25

Closed
aguschin opened this issue Sep 24, 2021 · 1 comment · Fixed by #181
Closed

Disambiguate passing multiple datasets to apply #25

aguschin opened this issue Sep 24, 2021 · 1 comment · Fixed by #181
Labels
api MLEM Python API cli MLEM command-line interface p3-low Low priority

Comments

@aguschin
Copy link
Contributor

@mike0sv comment:

it seems that this logic will apply model to multiple datasets in one run. My vision was to apply model once, but using multiple datasets (for example, .fit with X and y)

It seems we need to support both approaches. I had a case in mind when you have a folder with images and want to run your NN on all of them at once.

@aguschin aguschin transferred this issue from another repository Sep 30, 2021
@aguschin aguschin added cli MLEM command-line interface api MLEM Python API labels Sep 30, 2021
@aguschin aguschin added this to the Beta release milestone Oct 11, 2021
@aguschin aguschin added the p3-low Low priority label Nov 23, 2021
@mike0sv
Copy link
Contributor

mike0sv commented Mar 24, 2022

I decided to limit cli capabilities in favor of better UX. User can always call cli multiple times to do multiple infers, and regular models do not have multiple input datasets. For anything more complex api can be used

mike0sv added a commit that referenced this issue Mar 24, 2022
* reworked apply cli. limit to 1 dataset but allow on-the-fly importing
closes #25
closes #132

* update other cmds with new sexy meta decorator

* fast test fix
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api MLEM Python API cli MLEM command-line interface p3-low Low priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants