Skip to content

Conversation

@m3d
Copy link
Member

@m3d m3d commented Aug 25, 2021

This is the first step to get images from given places (where robot should see artifact, for example):

python subt\tools\locimg.py --loc 150.71 73.23 0.97 d:\\logs\aws\cave\C8\ver125\A600L.log --radius 10 --out-dir d:\logs\aws\cave\C8\ver125\tmpL\ --stream logimage_left.image

@m3d m3d requested review from jisa and tajgr August 25, 2021 13:39
Copy link
Collaborator

@jisa jisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess a v0 has to do something ...
What is the long term plan?


parser = argparse.ArgumentParser(description=__doc__)
parser.add_argument('logfile', help='recorded log file')
parser.add_argument('--out-dir', help='where tu dump relevant images')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/tu/to

from subt.trace import distance3D


POSE3D_STREAM = 'fromrospy.pose3d'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could also be a commandline flag.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK ... I just realized that we wanted to take this pose3d from true position ... so this is just first approximation.

@m3d
Copy link
Member Author

m3d commented Aug 25, 2021

I guess a v0 has to do something ...
What is the long term plan?

Long term you mean > 24h? ;) ... the plan was to help with "false negative" (?), i.e. artifacts we did not detect but in reality they were there and speed up analysis for @tajgr

p.s. at the moment it grabs some images near potential place (taken from state.tlog list of artifacts), but as we have 3 cameras on Pam is should reduce the number to much less number of images.

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

Successfully merging this pull request may close these issues.

2 participants