search

YuanGongND / ssast

Code for the AAAI 2022 paper "SSAST: Self-Supervised Audio Spectrogram Transformer".
BSD 3-Clause "New" or "Revised" License
365 stars 61 forks source link

Data Loading Logs #31

Open fabianbosshard opened 7 months ago

fabianbosshard commented 7 months ago

Hello Yuan

Thanks for the great work and for open-sourcing everything!

I have a quesiton about traintest_mask.py: Shouldn't end_time be updated after the evaluation? Because I think the way it is currently implemented leads to a very large number for per_sample_data_time direclty after the evaluation:

image

At first I thought it was because of cache or something similar, but I think it is because the evaluation time is also included in per_sample_data_time in the first iteration after the evaluation.

Best Regards, Fabian

YuanGongND commented 7 months ago

hi Fabian,

Thanks for pointing out this, it might be a bug, but it does not impact the performance, right? I just use this to check I/O and network bottleneck.

-Yuan

fabianbosshard commented 7 months ago

Thank you for the quick response. No don't worry, it has no impact on performance, the variable is onnly used to check I/O bottleneck, as you said!