-
Notifications
You must be signed in to change notification settings - Fork 33
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
Large scale training #17
Comments
@iejMac oh hey! yea, should be complete, probably one or two more bugs left to iron out i would stick with LFQ initially, as that was what the magvit2 paper proposed, although some people have reported better results with FSQ. i put it into one repository so we can test them against each other and find out |
Hi @iejMac I'd like to follow along if that's okay. It would be great if you could share any changes you make to the codebase to allow for larger scale training. I'm happy to share any weights I generate to help people get started with pretrained models. |
@jpfeil will do! Ok I think I'm mostly set up (had to port this code to a repo with a different style). My first question is - do we have some prepared configs (like what layers, how many frames, what fps etc.) which roughly correspond to some models they trained in the paper? Just so we can compare. For reference, currently I'm using the equivalent of this:
|
Oh I also noticed one thing - is there a reason we don't normalize the pixels before passing it into the model? Or did I just not catch where that's done? |
@lucidrains ah yeah ToTensor does but your VideoDataset doesn't do that and thats what I was using to test (was getting loss ~O(1e5)). magvit2-pytorch/magvit2_pytorch/data.py Line 159 in b2f105b
|
@iejMac shoot, i normalized for gifs, but not mp4s.. thank you Maciej! |
@iejMac yup, i can get some of the hyperparameters inline with the paper's probably Tuesday (currently in the middle of another project) |
cool, was just wondering if you have something on hand. I'll try to read/play around and I'll report here if I come up with something. Also for lowish-effort video dataloading video2numpy could be a good option! It's pretty fast and does all the normal preprocessing for you. Maybe I'll make a PR for that if you're interested. |
@iejMac would greatly appreciate it! 🙏 |
Following in this thread since it's also related to video loading: |
@mudtriangle there's a BGR format? 😅 |
Yep, and I think it's still default in cv2. |
@mudtriangle got it, put in a quick fix here (just doing it in tensor space, as i'm not familiar with cv2 enough) |
Hello, I'm wondering if there's any progress of hyperparameter/architecture config alignment with the magvit-v2 paper. |
Hey, wanted to start this comm channel as I'm looking to do a large scale training run using some of this code. I'm happy to share graphs/samples as I go along and wanted to ask a few things to start off:
As always, thanks for this! Very helpful
The text was updated successfully, but these errors were encountered: