ltgoslo / norbench

Natural language understanding benchmarks for Norwegian
MIT License
13 stars 4 forks source link

tsa trust_remote_code=True #37

Closed egilron closed 8 months ago

egilron commented 8 months ago

trust_remote_code=True seems to be needed now, for the config = AutoConfig.from_pretrained.
Hard-coding this seems not to be too dangerous.

akutuzov commented 8 months ago

It is needed only for models lacking the proper Transformers class (like LTG-Bert, for example). For all the rest, there's no need to use this (potentially very dangerous) parameter.

egilron commented 8 months ago

OK. Maybe I'll change the default model in stead then, so that the script runs with default settings.

akutuzov commented 8 months ago

May be it makes sense for now to simply put if/else statements checking whether the model has ltg/norbert3 in its name, and if yes, then turn on trust_remote_code=True?