Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

gluon-nlp/scripts/question_answering/train.py #1569

Open
rrjia opened this issue May 13, 2021 · 2 comments
Open

gluon-nlp/scripts/question_answering/train.py #1569

rrjia opened this issue May 13, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@rrjia
Copy link

rrjia commented May 13, 2021

Description

class ExponentialMovingAverage
in train.py the ema use two position:

if global_step == 1:
for name, param in model.collect_params().items():
ema.add(name, param.data(CTX[0]))

for name, param in model.collect_params().items():
ema(name, param.data(CTX[0]))

but model‘s params not update ,final we save model params as:

save model after train one epoch

    model.save_parameters(SAVE_MODEL_PREFIX_NAME +
                          time.asctime(time.localtime(time.time())))

is the ema not work?

@rrjia rrjia added the bug Something isn't working label May 13, 2021
@leezu
Copy link
Contributor

leezu commented May 13, 2021

There is no train.py in https://github.com/dmlc/gluon-nlp/tree/master/scripts/question_answering or https://github.com/dmlc/gluon-nlp/tree/v0.x/scripts/question_answering

Please provide a link to the code you're concerned about. Thanks

@rrjia
Copy link
Author

rrjia commented May 13, 2021

code link is : https://github.com/dmlc/gluon-nlp/blob/qanet/scripts/question_answering/train.py

appreciate for your supprot.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants