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

range argument to filter_poses() would be convenience #48

Open
kayvonf opened this issue Nov 17, 2017 · 0 comments
Open

range argument to filter_poses() would be convenience #48

kayvonf opened this issue Nov 17, 2017 · 0 comments

Comments

@kayvonf
Copy link

kayvonf commented Nov 17, 2017

When exploring the dataset, it's common to adjust the range of the pose results returned by filter_poses. Currently if you want to modify the default pose query range, you have to do this:

filtered = filter_poses('pose', hands_above_head, [Pose.LWrist, Pose.Nose, Pose.RWrist], Pose.objecfts.all().order_by('id').select_related('frame', 'frame__video')[0:250000:10])

To support the common case, it would be very convenient to support a named array argument to filter_poses(range=[min, max, stride]);

If you are worried about making it too easy to hang the server, add some error checking into filter_poses that errors out if the query requests a set of poses that is pretty large. (e.g., more than 2M).

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

No branches or pull requests

1 participant