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

Add DependencyViT #2062

Draft
wants to merge 82 commits into
base: main
Choose a base branch
from
Draft

Conversation

fffffgggg54
Copy link
Contributor

From-scratch impl of DependencyViT. Official impl not published. Not competitive with sota hierarchical models, advantage over isometric models is lost due to inability to use F.sdpa, but interesting mechanism. Mainly just ViT with a different attn/block, but the cumulative m and topk pruning makes stuff a bit messy. Block exploits nn.Sequential's untyped intermediate states and feeds tuples through for the cumulative m calculation. I messed around a bit with building dependency trees using the dependency masks, but I have no clue what I'm doing. Currently training weights, have a [email protected]% and playing around with a few implementation details and training recipe. Contacted @dingmyu for reference code and weights but no response yet.

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@Smartappli
Copy link

@fffffgggg54 is this still in progress ?

@fffffgggg54
Copy link
Contributor Author

@fffffgggg54 is this still in progress ?

This model is on hold for me. It requires several additional deviations from the standard ViT impl that I didn't realize were a part of the model. I will probably take a look at it after I get back from vacation. The current impl can be trained to around 73.X% top-1, around 2% below the reported top-1.

@Smartappli
Copy link

Not yet teady for review?

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.

3 participants