Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update default model names #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bzz
Copy link

@bzz bzz commented Apr 29, 2020

Some models have a _new prefix now:

$ curl 'https://www.inf.unibz.it/~hbabii/pretrained_models_list'

dev_10k_1_10_190923.132328_new
langmodel-large-split_10k_2_1024_191007.112241_-_langmodel-large- 
split_10k_2_1024_191022.141344_new
langmodel-small-split_10k_1_512_190906.154943_new
langmodel-large-split_10k_1_512_190926.120146_new
langmodel-small-split-reversed_10k_1_512_200117.095729

Otherwise, webapp gets HTTP 500 request from the backend on looking up non-existant models by name:

Full stack trace
2020-04-28 19:01:20,390 [werkzeug] INFO: 127.0.0.1 - - [28/Apr/2020 19:01:20] "POST /api/languagemodel HTTP/1.1" 500 -
Traceback (most recent call last):
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/app.py", line 2463, in __call__
    return self.wsgi_app(environ, start_response)
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/app.py", line 2449, in wsgi_app
    response = self.handle_exception(e)
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask_cors/extension.py", line 161, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/app.py", line 1866, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/app.py", line 2446, in wsgi_app
    response = self.full_dispatch_request()
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/app.py", line 1951, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask_cors/extension.py", line 161, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/app.py", line 1820, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/app.py", line 1949, in full_dispatch_request
    rv = self.dispatch_request()
  File "~/gigacode/.venv/lib/python3.7/site-packages/flask/app.py", line 1935, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "~/gigacode/lm-powered-backend/webservice.py", line 67, in api_languagemodel
    return entropy_controller.get_entropies(request, models)
  File "~/gigacode/lm-powered-backend/controller/entropy_controller.py", line 198, in get_entropies
    selected_model = models[selected_model_name]
KeyError: 'langmodel-large-split_10k_1_512_190926.120146'

Some models have a _new prefix now:

```
$ curl 'https://www.inf.unibz.it/~hbabii/pretrained_models_list'
dev_10k_1_10_190923.132328_new
langmodel-large-split_10k_2_1024_191007.112241_-_langmodel-large-split_10k_2_1024_191022.141344_new
langmodel-small-split_10k_1_512_190906.154943_new
langmodel-large-split_10k_1_512_190926.120146_new
langmodel-small-split-reversed_10k_1_512_200117.095729
```

Signed-off-by: Alexander Bezzubov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant