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.

Sign up for GitHub

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

Jump to bottom

Profiling #139

Merged
merged 5 commits into from
Feb 13, 2024
Merged

Profiling #139

merged 5 commits into from
Feb 13, 2024

Conversation

wensi-ai
Copy link
Contributor

Add profiling scripts

@wensi-ai wensi-ai changed the base branch from main to og-develop March 30, 2023 18:09
@wensi-ai wensi-ai force-pushed the profiling branch 6 times, most recently from 343bac5 to 2dc9c41 Compare April 5, 2023 00:27
@wensi-ai wensi-ai force-pushed the profiling branch 7 times, most recently from 1dfd8dd to 2ce0bfc Compare April 11, 2023 04:41
@wensi-ai wensi-ai force-pushed the profiling branch 3 times, most recently from 711f821 to ff53617 Compare April 11, 2023 19:47
@wensi-ai wensi-ai changed the title Profiling [WIP] Profiling Apr 11, 2023
@wensi-ai wensi-ai marked this pull request as ready for review April 11, 2023 20:11
@wensi-ai wensi-ai closed this Apr 11, 2023
@wensi-ai wensi-ai reopened this Apr 11, 2023
@wensi-ai wensi-ai force-pushed the profiling branch 2 times, most recently from a3374c0 to 0dfc6ed Compare April 11, 2023 20:15
@wensi-ai wensi-ai marked this pull request as draft April 11, 2023 20:15
@wensi-ai wensi-ai force-pushed the profiling branch 4 times, most recently from 9ca2c2d to 845058b Compare April 11, 2023 20:45
@wensi-ai wensi-ai marked this pull request as ready for review February 6, 2024 00:27
@wensi-ai wensi-ai force-pushed the profiling branch 2 times, most recently from 88c46ac to 1cf1243 Compare February 6, 2024 04:28
@wensi-ai wensi-ai changed the title [WIP] Profiling Profiling Feb 6, 2024
wensi-ai
wensi-ai commented Feb 6, 2024
View reviewed changes
omnigibson/simulator.py Outdated Show resolved Hide resolved
cgokmen
cgokmen approved these changes Feb 6, 2024
View reviewed changes
omnigibson/macros.py Outdated Show resolved Hide resolved
@cgokmen
Copy link
Contributor

cgokmen commented Feb 6, 2024

LGTM. I'm really excited about this feature. Thanks a lot for doing all the diagnostic work!

@wensi-ai wensi-ai force-pushed the profiling branch 4 times, most recently from bfc6a98 to 1d689bb Compare February 10, 2024 01:41
@wensi-ai wensi-ai force-pushed the profiling branch 3 times, most recently from 6cdefd3 to 3c5596d Compare February 12, 2024 20:04
ChengshuLi
ChengshuLi approved these changes Feb 12, 2024
View reviewed changes
omnigibson/prims/entity_prim.py Outdated Show resolved Hide resolved
@ChengshuLi ChengshuLi enabled auto-merge February 13, 2024 20:26
@ChengshuLi ChengshuLi merged commit e6419d1 into og-develop Feb 13, 2024
4 checks passed
@ChengshuLi ChengshuLi deleted the profiling branch February 13, 2024 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ChengshuLi ChengshuLi ChengshuLi approved these changes

@cgokmen cgokmen cgokmen approved these changes

@cremebrule cremebrule Awaiting requested review from cremebrule

Assignees

@wensi-ai wensi-ai

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Finish setting up speed benchmark Merge benchmark PR
4 participants
@wensi-ai @cgokmen @ChengshuLi @cremebrule