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

Specify Important Edges #29

Open
JarredAllen opened this issue Jul 19, 2019 · 7 comments
Open

Specify Important Edges #29

JarredAllen opened this issue Jul 19, 2019 · 7 comments
Labels
low-priority Not very important

Comments

@JarredAllen
Copy link
Contributor

For ROIs, we need some way to specify which edges are relevant and could have ants crossing them.

The current plan is to go into roimodify.py and add a tool for selecting the important edges for each ROI. Then, in the edgefromtrack.py script, we need to modify the file to only allow ants to enter and exit from one such edge, if there's one.

@JarredAllen
Copy link
Contributor Author

Specifying the important edges has been handled as of ea3a27d, but no automatic detection is in place.

The code still doesn't do anything with the values, either.

@mdonaldsonmatasci
Copy link
Member

Not quite sure what you meant by automatic detection, but I realized you could by default assume that in a rectangle the short edges are the important ones, and in any other shape assume all edges are important? This would probably get it right most of the time.

@JarredAllen
Copy link
Contributor Author

@mdonaldsonmatasci That's what I meant by automatic detection.

@mdonaldsonmatasci
Copy link
Member

ok sounds good

@JarredAllen
Copy link
Contributor Author

As of 9b4827f, the code now uses the important edges and classifies tracks as going from one important edge to another important edge only, if a user specifies them. Because specifying the important edges is easy to do, I'm going to call enabling the code to automatically guess at which edges are important to be a low-priority concern.

@JarredAllen JarredAllen added low-priority Not very important and removed priority labels Jul 23, 2019
@mdonaldsonmatasci
Copy link
Member

mdonaldsonmatasci commented Jul 23, 2019 via email

@JarredAllen
Copy link
Contributor Author

@mdonaldsonmatasci I'll look into reusing manually specified ROIs in a bit, but my gut feeling is that it should be possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low-priority Not very important
Projects
None yet
Development

No branches or pull requests

2 participants