Skip to content

Tweaks #15

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

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Tweaks #15

wants to merge 11 commits into from

Conversation

vijayiyer05
Copy link
Collaborator

  • Tweaks to headings/variable names/comments/etc.
    -Stages vectorization of crop computation in Postprocess1CustomLayer which may increase performance somewhat; currently commented out, to be considered for future PR after further benchmarking

vijayiyer05 and others added 11 commits November 6, 2022 21:10
…orary script for performance comparison purposes
…llow focus on frame-by-frame inference performance
Vectorized Nested For Loop in Postprocess1CustomLayer
* Use more descriptive names (include 'keypoint') for keypoint prediction variables
* Standardize heading capitalization style
* Prefer camelCase 
* Prefer X_ for intermediate vars
* Add comments to batch prediction display for loop
* Use f1/f2 figure handle variable names (matches MATLAB documentation pattern)
To be revisited in a later atomic pull request if performance (or other) gains are validated.
@vijayiyer05 vijayiyer05 requested a review from talmo August 21, 2023 00:29
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