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

[Feature] support mim #717

Merged
merged 2 commits into from
Jul 27, 2021
Merged

[Feature] support mim #717

merged 2 commits into from
Jul 27, 2021

Conversation

xiexinch
Copy link
Collaborator

  • add MIM introduction in README
  • modify setup.py to support mim as mmseg entry point

@xiexinch
Copy link
Collaborator Author

@zhouzaida

@codecov
Copy link

codecov bot commented Jul 22, 2021

Codecov Report

Merging #717 (dab0413) into master (4d34581) will not change coverage.
The diff coverage is n/a.

❗ Current head dab0413 differs from pull request most recent head ee62c53. Consider uploading reports for the commit ee62c53 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #717   +/-   ##
=======================================
  Coverage   85.28%   85.28%           
=======================================
  Files         107      107           
  Lines        5817     5817           
  Branches      952      952           
=======================================
  Hits         4961     4961           
  Misses        673      673           
  Partials      183      183           
Flag Coverage Δ
unittests 85.26% <ø> (ø)

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


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d34581...ee62c53. Read the comment docs.

MANIFEST.in Outdated
include mmseg/.mim/model-index.yml
recursive-include mmseg/.mim/configs *.py *.yml
recursive-include mmseg/.mim/tools *.py *.sh
recursive-include mmseg/.mim/demo *.py
Copy link
Member

Choose a reason for hiding this comment

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

This line should be removed

setup.py Outdated
else:
return

filenames = ['tools', 'configs', 'demo', 'model-index.yml']
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
filenames = ['tools', 'configs', 'demo', 'model-index.yml']
filenames = ['tools', 'configs', 'model-index.yml']

Copy link
Member

@zhouzaida zhouzaida left a comment

Choose a reason for hiding this comment

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

LGTM

@Junjun2016 Junjun2016 merged commit 6526593 into open-mmlab:master Jul 27, 2021
bowenroom pushed a commit to bowenroom/mmsegmentation that referenced this pull request Feb 25, 2022
* support mim

* remove mim demo
aravind-h-v pushed a commit to aravind-h-v/mmsegmentation that referenced this pull request Mar 27, 2023
* Avoid negative strides for tensors

* Changed not to make torch.tensor

* Removed a needless copy
wjkim81 pushed a commit to wjkim81/mmsegmentation that referenced this pull request Dec 3, 2023
sibozhang pushed a commit to sibozhang/mmsegmentation that referenced this pull request Mar 22, 2024
* resolve comments

* update changelog

* cn doc for demo

* refine

* refine

* refine

* refine

* refine

* refine
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.

None yet

3 participants