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 some URLs of RAFT(MMFlow 1.0) #206

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

Zachary-66
Copy link
Collaborator

Motivation

The config raft_8x2_50k_kitti2015_288x960 related URLs in README.md and metafile.yml are wrong, which may cause confusions.

Modification

  1. configs/raft/README.md
  2. configs/raft/metafile.yml
  3. configs/liteflownet/metafile.yml

TODO

Rename the files in cloud:

  1. raft_8x2_50k_kitti2015_368x768.log.json -> raft_8x2_50k_kitti2015_288x960.log.json
  2. raft_8x2_50k_kitti2015_368x768.py -> raft_8x2_50k_kitti2015_288x960.py
  3. raft_8x2_50k_kitti2015_368x768.pth -> raft_8x2_50k_kitti2015_288x960.pth

@codecov
Copy link

codecov bot commented Aug 5, 2022

Codecov Report

Merging #206 (30a1f4e) into master (1105f5c) will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #206      +/-   ##
==========================================
+ Coverage   68.89%   69.02%   +0.13%     
==========================================
  Files          93       93              
  Lines        5195     5195              
  Branches      741      741              
==========================================
+ Hits         3579     3586       +7     
+ Misses       1502     1495       -7     
  Partials      114      114              
Flag Coverage Δ
unittests 69.02% <ø> (+0.13%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
mmflow/datasets/pipelines/transforms.py 78.66% <0.00%> (+1.16%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@MeowZheng MeowZheng merged commit 47d7ece into open-mmlab:master Aug 8, 2022
@Zachary-66 Zachary-66 deleted the fix_urls_1 branch August 10, 2022 08:06
# 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.

2 participants