Skip to content

Why did you comment this line? #63

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

Closed
DecaYale opened this issue Nov 14, 2024 · 1 comment
Closed

Why did you comment this line? #63

DecaYale opened this issue Nov 14, 2024 · 1 comment

Comments

@DecaYale
Copy link

Hi, I found line 156 in run.py generated wrong masks on my dataset while the depth maps are reasonable. Did you have similar issues before? I noticed that you have commented this line in your script. Is the masking procedure adopted in your paper? Thank you!

# depth[mask == 0] = 0

for name, mask in tqdm(zip(names, masks)):
        depth_bin_path = f'{target}/dense/stereo/depth_maps/{name}.geometric.bin'
        if not os.path.exists(depth_bin_path):
            continue
        depth_fname = depth_bin_path
        depth = read_array(depth_fname)
        # depth[mask == 0] = 0
        np.save(f'{target}/depth_colmap/{name[:-4]}.npy', depth)
@DecaYale DecaYale changed the title Why do you commented this line? Why did you comment this line? Nov 14, 2024
@ghy0324
Copy link
Member

ghy0324 commented Nov 26, 2024

Hi! Thank you for your interest in our work!

Our original goal was to generate masks to retain only the depth values of pixels that are kept during the fusion process. However, we later discovered that the order of the masks are disrupted. As a result, we commented this line. In fact, whether this line is commented or not has little impact on the results, as the current masks behave like randomly masking due to the wrong ordering. If the correct order could be restored, the final results might potentially be improved using the masks.

Welcome for further discussion.

@ghy0324 ghy0324 closed this as completed Nov 26, 2024
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

No branches or pull requests

2 participants