-
Notifications
You must be signed in to change notification settings - Fork 2
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
Subject Tracking Workflow #204
Conversation
Yun-Wu
commented
Aug 23, 2024
- add subject tracking spec
- generated
- trajectory map
- Add mean speed single value
- max speed
- Add time density map
- generated files
- dashboard
Thank you @cisaacstern for fixing the test! The pr is ready for review now! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Yun-Wu looks great TYSM!
I realized that we were missing a test for the async compilation so I've made a PR to this PR with that fix: #213. Assuming tests pass there, can we merge that here before this goes in?
Note, however, that the e2e tests for the subject tracking workflow did run (and pass) already in your PR:
My PR just adds a check for adding the compiled artifact to the repo.