-
Notifications
You must be signed in to change notification settings - Fork 0
Profiling example #4
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
base: main
Are you sure you want to change the base?
Conversation
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.
Pull Request Overview
This PR sets up experiments for profiling the Arc Unwrapping algorithm using both loop‐based and Dask-based methods, as described in issue #88. Key changes include:
- Adding a Dask-based unwrapping implementation (both script and notebook versions) to leverage parallel processing.
- Including a loop-based unwrapping implementation (script and notebook) for baseline comparison.
- Updating documentation in the README to describe profiling procedures and commands.
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
File | Description |
---|---|
profiling_example/lambda_unwrap_dask.py | Adds a Dask-based parallel unwrapping script with process‐based scheduling. |
profiling_example/lambda_unwrap_dask.ipynb | Introduces a Jupyter notebook for debugging/visualizing the Dask-based implementation. |
profiling_example/lambda.py | Provides a loop-based unwrapping script for direct unwrapping execution. |
profiling_example/lambda.ipynb | Contains a notebook version of the loop-based unwrapping method for analysis. |
profiling_example/README.md | Updates documentation to explain profiling experiments using both approaches. |
Co-authored-by: Copilot <[email protected]>
Hi @fnattino, in this PR I setup the profiling experiment. The explaination of the files should be in README, including a link to the results. I put the discussion of the results in this discussion channel . Can you review on the profiling method? Thanks |
Set up experiments for profiling, as required in https://github.com/TUDelftGeodesy/DePSI_group/issues/88