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

Support input of non three-channel image #1866

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Conversation

MGAMZ
Copy link
Contributor

@MGAMZ MGAMZ commented Jan 16, 2024

Motivation

Current Implementation does not support images with non 3-channels

Modification

Add in_chans augment and pass it to PatchEmbed layer.

Add in_chans augment and pass it to PatchEmbed layer.
Copy link

codecov bot commented Jan 16, 2024

Codecov Report

Attention: 1074 lines in your changes are missing coverage. Please review.

Comparison is base (896964b) 65.16% compared to head (f378f36) 62.77%.
Report is 17 commits behind head on dev.

Files Patch % Lines
mmpretrain/models/multimodal/ram/bert.py 0.00% 449 Missing ⚠️
mmpretrain/models/multimodal/ram/ram.py 0.00% 144 Missing ⚠️
mmpretrain/models/multimodal/clip/clip.py 0.00% 139 Missing ⚠️
mmpretrain/models/multimodal/ram/gradio_demo.py 0.00% 54 Missing ⚠️
mmpretrain/models/multimodal/llava/modules.py 0.00% 44 Missing ⚠️
mmpretrain/models/multimodal/ram/openset_utils.py 0.00% 42 Missing ⚠️
...retrain/models/multimodal/clip/clip_transformer.py 0.00% 41 Missing ⚠️
mmpretrain/models/multimodal/ram/utils.py 0.00% 38 Missing ⚠️
mmpretrain/models/multimodal/minigpt4/minigpt4.py 0.00% 28 Missing ⚠️
mmpretrain/datasets/minigpt4_dataset.py 0.00% 27 Missing ⚠️
... and 12 more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #1866      +/-   ##
==========================================
- Coverage   65.16%   62.77%   -2.39%     
==========================================
  Files         362      375      +13     
  Lines       26182    27177     +995     
  Branches     4165     4309     +144     
==========================================
  Hits        17061    17061              
- Misses       8501     9495     +994     
- Partials      620      621       +1     
Flag Coverage Δ
unittests 62.77% <0.27%> (-2.39%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MGAMZ
Copy link
Contributor Author

MGAMZ commented Apr 24, 2024

Github Shows only 1 file changed, but codecov shows many files have been changed which is not the truth.

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

1 participant