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

about the ar loop? #7

Open
azraelkuan opened this issue Dec 28, 2021 · 2 comments
Open

about the ar loop? #7

azraelkuan opened this issue Dec 28, 2021 · 2 comments

Comments

@azraelkuan
Copy link

from the code:

signals[start:start + cargan.CHUNK_SIZE] += signal.squeeze()

for each chunk output samples, it will be add to signals,
but in the for loop,

for i in range(0, features.shape[2] - feat_chunk + 1, feat_hop):

we have the feat_hop,,
for my understand, it will cumsum on the signals, but we only need the first feat_hop * hop_size samples, right?

@azraelkuan
Copy link
Author

there is boundary mismatch between different chunks, how to solve this?

@maxrmorrison
Copy link
Contributor

The code is written to allow the hop size and chunk size to be different. But if you check their values, they are the same. I experimented with a smaller hop size for a while, but found hop size == chunk size to be best.

As mentioned in the conclusion of our paper, we don't yet know how to solve the boundary artifacts. However, our experiments indicate that these artifacts are less severe than the periodicity artifacts.

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

No branches or pull requests

2 participants