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

Augmented tensor: labels renamed by Child + default args when adding one node #111

Merged
merged 10 commits into from
Oct 19, 2021

Conversation

thibo73800
Copy link
Contributor

@thibo73800 thibo73800 commented Oct 18, 2021

In this merge request:

  • Rename label in augmented tensor by child & children (datasets samples need to be updated)
  • Default mergeable & align_dim when adding node

Example

import aloscene
import torch

frame = aloscene.Frame("/path/to/image.jpg")

vector = torch.as_tensor([0, 1, 2])
# align on T & B by defautl & mergeable
vector = aloscene.tensors.AugmentedTensor(vector, names=(None,))
frame.add_child("vector", vector)

frame2 = frame.resize((200, 200))
frame = frame.batch_list([frame, frame2])
frame.get_view().render()

@ragier
@jsalotti
@Johansmm
@LucBourrat1

@Johansmm Can you try out the changes on your current project and let me know the errors you stil lhave (with some simple code to reproduce)

@thibo73800 thibo73800 added the aloscene aloscene label Oct 18, 2021
@thibo73800 thibo73800 self-assigned this Oct 18, 2021
@ragier
Copy link
Contributor

ragier commented Oct 18, 2021

Consider using child/parent semantic instead of node.

@thibo73800 thibo73800 changed the title Augmented tensor: labels renamed by Node + default args when adding one node Augmented tensor: labels renamed by Child + default args when adding one node Oct 18, 2021
@thibo73800 thibo73800 merged commit 0831533 into master Oct 19, 2021
@thibo73800 thibo73800 deleted the add-node-api branch October 19, 2021 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aloscene aloscene
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants