Skip to content

fix: issues 1418, 1420#1549

Open
Ihab-Asaad wants to merge 2 commits intodevfrom
issues-1418/1420
Open

fix: issues 1418, 1420#1549
Ihab-Asaad wants to merge 2 commits intodevfrom
issues-1418/1420

Conversation

@Ihab-Asaad
Copy link
Copy Markdown
Contributor

@Ihab-Asaad Ihab-Asaad commented Apr 8, 2022

Resolves #1418, #1420

Comment thread deeppavlov/core/trainers/nn_trainer.py Outdated
Comment on lines +307 to +308
# can't specify save path for chainer from here!
# have to duplicate code from chainer.save
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to address these comments in PR.

@IgnatovFedor IgnatovFedor changed the title fix issues 1418,1420 fix: issues 1418, 1420 Apr 18, 2022
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.

2 participants