keras-team / keras-cv

Industry-strength Computer Vision workflows with Keras
Other
1.01k stars 331 forks source link

Add MAXIM model #924

Open innat opened 2 years ago

innat commented 2 years ago

Short Description

MAXIM: Multi-Axis MLP for Image Processing: I think it's a follow-up work of MaxViT from google. It shows a great performance on the following low-vision task, i.e. for OCR.

image3

It looks useful for real world problem (blurry, haze, rain-drop). Such problems are very acute for vision project. So, such solutions might weight quick adaptation of kcv in industrial projects (alike).

Papers

Existing Implementations

cc. @vztu

tanzhenyu commented 2 years ago

Open this up for contribution

innat commented 2 years ago

cc @sayakpaul

IMvision12 commented 1 year ago

I'm working on integrating the maxim model and referencing the original flax implementation. I have one question: do we need weights as well? @tanzhenyu

tanzhenyu commented 1 year ago

I'm working on integrating the maxim model and referencing the original flax implementation. I have one question: do we need weights as well? @tanzhenyu

Yes, we need the weights. The development cycle would be similar to ViT: https://github.com/keras-team/keras-cv/pull/1014, add the model and port the weights, also add the conversion scripts for porting the weights

bhack commented 1 year ago

Yes, we need the weights. The development cycle would be similar to ViT: https://github.com/keras-team/keras-cv/pull/1014, add the model and port the weights, also add the conversion scripts for porting the weights

I hope we could standardize a bit the model contribution process after we will handle:

https://github.com/keras-team/keras-cv/discussions/954

IMvision12 commented 1 year ago

I'm working on integrating the maxim model and referencing the original flax implementation. I have one question: do we need weights as well? @tanzhenyu

Yes, we need the weights. The development cycle would be similar to ViT: #1014, add the model and port the weights, also add the conversion scripts for porting the weights

Okay will add the conversion script too!!

innat commented 1 year ago

@IMvision12 Have you checked ? https://github.com/sayakpaul/maxim-tf

IMvision12 commented 1 year ago

@IMvision12 Have you checked ? https://github.com/sayakpaul/maxim-tf

Oh we have tf implementation for maxim, because when I saw on paper with code there was only official flax implementation will refer that too thx for the info @innat. Also as we already have weights ported to TF and uploaded by @sayakpaul on TF-Hub do we still require the conversion script? @tanzhenyu

github-actions[bot] commented 1 month ago

This issue is stale because it has been open for 180 days with no activity. It will be closed if no further activity occurs. Thank you.