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

mergeable properties for augmented tensors #241

Open
tflahaul opened this issue Nov 16, 2022 · 0 comments
Open

mergeable properties for augmented tensors #241

tflahaul opened this issue Nov 16, 2022 · 0 comments
Assignees
Labels
aloscene aloscene discussion enhancement New feature or request hard

Comments

@tflahaul
Copy link
Contributor

tflahaul commented Nov 16, 2022

As of today we can only merge tensors that have the exact same properties. Trying to merge tensors with different .projection or .distortion will raise

AssertionError: Trying to merge augmented tensor with different property: projection

Being able to merge these properties in lists would be great.

Expected return of a torch.cat((a, b, c), dim=0) with a, b & c being tensors with different projection properties:

tensor(
	projection=['kumler_bauer', 'equidistant', 'pinhole']
	normalization=minmax_sym
	pose=torch.Size([3, 4, 4])
	mask=torch.Size([3, 1, H, W])
	depth=torch.Size([3, 1, H, W])
	cam_intrinsic=torch.Size([3, 4, 4])
	cam_extrinsic=torch.Size([3, 4, 4])
	distortion=[[0.45, 0.39, 0.12], 0.25, 1.0]
@tflahaul tflahaul added enhancement New feature or request aloscene aloscene labels Nov 16, 2022
@tflahaul tflahaul self-assigned this Nov 17, 2022
tflahaul added a commit that referenced this issue Nov 17, 2022
merge aten properties when they're different
@tflahaul tflahaul linked a pull request Nov 17, 2022 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aloscene aloscene discussion enhancement New feature or request hard
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant