executorch
executorch copied to clipboard
[#9971] Gracefully error out in ETDump part 3 for *profiling_delegate
Summary
Gracefully throw error for
start_profiling_delegate end_profiling_delegate end_profiling_delegate
- Changed the method signature to Result<>
- Use ET_CHECK_OR_RETURN_ERROR next: check_ready_to_add_event will need to come in later PRs
Test plan
test/run_oss_cpp_tests.sh
Not sure how to trigger CI in draft mode
@Gasoonjia @zhenyan-zhang-meta
:link: Helpful Links
:test_tube: See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/10147
- :page_facing_up: Preview Python docs built from this PR
Note: Links to docs will display an error until the docs builds have been completed.
:white_check_mark: No Failures
As of commit a47a195f532b83e4914a2175f78835b686e0fc63 with merge base e84c319665e37e5d730953b2ffbe23c4ae41858b ():
:green_heart: Looks good so far! There are no failures yet. :green_heart:
This comment was automatically generated by Dr. CI and updates every 15 minutes.
@pytorchbot label "topic: not user facing"
Also please solve the ci issue.
Not sure how to trigger CI in draft mode
It has to be from us to manually trigger it for you. I've just done it.
Could you also regularly merge the latest changes from main into this branch?
@zhenyan-zhang-meta I've merged this onto the latest main branch 15 minutes ago so it should be ok now.
Also please solve the ci issue.
Most of errors are related to huggingface. I guess they are temporary? Please feel free to suggest how to reproduce it locally here. Thanks
OSError: We couldn't connect to 'https://huggingface.co' to load this file, couldn't find it in the cached files and it looks like google/mobilebert-uncased is not the path to a directory containing a file named config.json.
@Gasoonjia has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
Please let me merge it -- there're some internal issue i need to fix
Please let me merge it -- there're some internal issue i need to fix
@Gasoonjia Thanks!. Do you need me to close this PR and wait for you?
@pytorchbot label "release notes: none"
@Gasoonjia please help merge this
Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.Stale pull requests will automatically be closed after 30 days of inactivity.
Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.Stale pull requests will automatically be closed after 30 days of inactivity.