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

train bug #37

Open
zpz915 opened this issue Jul 19, 2024 · 14 comments · May be fixed by #40
Open

train bug #37

zpz915 opened this issue Jul 19, 2024 · 14 comments · May be fixed by #40

Comments

@zpz915
Copy link

zpz915 commented Jul 19, 2024

python train.py \

--dataset mpiigaze
--snapshot output/snapshots
--gpu 0
--num_epochs 50
--batch_size 16
--lr 0.00001
--arch ResNet101
--alpha 1
Loading data.
45000 items removed from dataset that have an angle > 0
Traceback (most recent call last):
File "train.py", line 276, in
train_loader_gaze = DataLoader(
File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 268, in init
sampler = RandomSampler(dataset, generator=generator)
File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/sampler.py", line 102, in init
raise ValueError("num_samples should be a positive integer "
ValueError: num_samples should be a positive integer value, but got num_samples=0

@zpz915
Copy link
Author

zpz915 commented Jul 19, 2024

i read the code,find dataset=Mpiigaze(testlabelpathombined,args.gazeMpiimage_dir, transformations, True, fold) give 5 paramters ,but class Mpiigaze(Dataset):
def init(self, pathorg, root, transform, train, angle,fold=0): accept 6 paramters ,is there any bug?

@zpz915
Copy link
Author

zpz915 commented Jul 19, 2024

when i guess angel=180,the train also meet other bug:
Traceback (most recent call last):
File "train.py", line 299, in
{'params': get_ignored_params(model, args.arch), 'lr': 0},
TypeError: get_ignored_params() takes 1 positional argument but 2 were given

@yakhyo
Copy link

yakhyo commented Aug 5, 2024

hi @zpz915 , have you succeeded training the model. I am also facing the same issue.

@forlayo
Copy link

forlayo commented Aug 19, 2024

Same issue here, any solutions ?

@yakhyo
Copy link

yakhyo commented Aug 20, 2024

@forlayo , I succeeded training Gaze360 and going to release it soon and adding mobilenet new backbone as well.

@tiamo405
Copy link

I tried to download the dataset but it seems like the dataset is too big, can anyone split the dataset for me around 5gb so I can download it?

@yakhyo
Copy link

yakhyo commented Aug 23, 2024

@tiamo405 , MPIIFaceGaze is relatively smaller than Gaze360.

@tiamo405
Copy link

@tiamo405
Copy link

@ipsampling
Copy link

I have identified and resolved issues in the code, and understand why using the Gaze360 dataset can result in successful training while using the MPIIFaceGaze dataset reports errors. This is because MPIIFaceGaze data processing lacks an angle parameter, and all data with angles less than 0 degrees are filtered out.

@NuoZ
Copy link

NuoZ commented Dec 6, 2024

我已经识别并解决了代码中的问题,并了解了为什么使用 Gaze360 数据集可以成功训练,而使用 MPIIFaceGaze 数据集报告错误。这是因为 MPIIFaceGaze 数据处理缺少 angle 参数,所有角度小于 0 度的数据都会被过滤掉。

Hello, so how did you solve the problem, just delete the parameter or add the code to handle the angle parameter?

@yakhyo
Copy link

yakhyo commented Dec 6, 2024

@NuoZ , i resolved the problem and reproduced some of the results.

@ipsampling
Copy link

我已经识别并解决了代码中的问题,并了解了为什么使用 Gaze360 数据集可以成功训练,而使用 MPIIFaceGaze 数据集报告错误。这是因为 MPIIFaceGaze 数据处理缺少 angle 参数,所有角度小于 0 度的数据都会被过滤掉。

您好,那么您是怎么解决问题的,只是删除参数还是添加代码来处理 angle 参数呢?
like this dataset=Mpiigaze(testlabelpathombined,args.gazeMpiimage_dir, transformations, True,180, fold)

shojint added a commit to shojint/L2CS-Net that referenced this issue Jan 1, 2025
shojint added a commit to shojint/L2CS-Net that referenced this issue Jan 1, 2025
@shojint shojint linked a pull request Jan 1, 2025 that will close this issue
@shojint
Copy link

shojint commented Jan 1, 2025

Sorry. This is my first pull request, so I was a bit clumsy. Please feel free to look at my commit.

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 a pull request may close this issue.

7 participants