Skip to content
New issue

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

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

Already on GitHub? # to your account

[Fix] Fix bug: Resume can not work. #820

Merged
merged 1 commit into from
Apr 19, 2021
Merged

Conversation

kennymckormick
Copy link
Member

The first line of cfg.text is the config filename, which will not be parsed by this line:
https://github.com/open-mmlab/mmcv/blob/de4f14e9cd547fb8bf886a362ad4f02a0be95141/mmcv/runner/base_runner.py#L346

Use cfg.pretty_text instead.

The first line of cfg.text is the config filename, which will not be parsed by this line:
https://github.com/open-mmlab/mmcv/blob/de4f14e9cd547fb8bf886a362ad4f02a0be95141/mmcv/runner/base_runner.py#L346

Use cfg.pretty_text instead.
@innerlee innerlee merged commit c96ec7d into master Apr 19, 2021
@dingfengshi
Copy link

你好,我修改成pretty_text以后还是不能resume, 还是识别不了第一行

kennymckormick added a commit that referenced this pull request Apr 20, 2021
innerlee pushed a commit that referenced this pull request Apr 20, 2021
@kennymckormick
Copy link
Member Author

你好,我修改成pretty_text以后还是不能resume, 还是识别不了第一行

My bad, I didn't replace it completely, you also need to replace text in cfg.checkpoint_config.meta by pretty_text. I have raised a patch (#824) for it.
Besides, you can resume from a ckpt only if the ckpt is trained with pretty_text rather than text.

@kennymckormick kennymckormick deleted the kennymckormick-patch-1 branch May 22, 2021 09:24
Naoki-Wake added a commit to Naoki-Wake/mmaction2 that referenced this pull request Jul 14, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants