webis-de / small-text

Active Learning for Text Classification in Python
https://small-text.readthedocs.io/
MIT License
547 stars 60 forks source link

Issue 36 fix #38

Closed JP-SystemsX closed 1 year ago

JP-SystemsX commented 1 year ago

Addressing Issue #36

The _get_param function of the finetuning arguments should now:

Still has problems with:

chschroeder commented 1 year ago

After the discussion, I think this solution is the right way. Thanks!

Do you have any pending tasks here? Otherwise I would just merge it.

After that I would like to add 1-2 unit tests (which were apparently lacking in this area of the code), and afterwards it should be done.

chschroeder commented 1 year ago

@JP-SystemsX See the comment above regarding your question in #36 on how to proceed.