Skip to content

Update requirements.txt, make timm version flexible #284

Open
bonlime wants to merge 1 commit intosalesforce:mainfrom
bonlime:patch-1
Open

Update requirements.txt, make timm version flexible #284
bonlime wants to merge 1 commit intosalesforce:mainfrom
bonlime:patch-1

Conversation

@bonlime
Copy link
Copy Markdown

@bonlime bonlime commented May 10, 2023

I'm running into troubles trying to add this package as requirement using poetry, since some of other packages in my lib require higher versions of timm. I tested that your library works with higher versions of timm library

@dxli94 review please

Make timm version flexible
@tyler-romero
Copy link
Copy Markdown

tyler-romero commented May 17, 2023

I also would like for the timm version to be flexible (or up to date, timm just released a v0.9)

@danieltudosiu
Copy link
Copy Markdown

@bonlime With what versions of timm is lavis compatible from your experience?

@bonlime
Copy link
Copy Markdown
Author

bonlime commented Jun 5, 2023

@danieltudosiu For me it works with 0.6.13

@xinyiliu128
Copy link
Copy Markdown

any plans to merge this pr? I also want to work with a higher version of timm package.

@bonlime
Copy link
Copy Markdown
Author

bonlime commented Jul 26, 2023

@dxli94 Hey, could you please review this?

@siddk
Copy link
Copy Markdown

siddk commented Sep 4, 2023

Bumping this - is there a reason to pin timm to 0.4.12?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants