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

Remove analysis_utils.py #506

Merged

jeremykubica merged 1 commit into main from analysis_utils

Mar 5, 2024

Contributor

jeremykubica commented Mar 5, 2024

Removes analysis_utils.py by moving the batch loading into run_search.py. Includes a few small changes to clean up the interface including:

Set everything in the loading function by a SearchConfiguration object instead of a combination of object attributes and function parameters.
Change do_gpu_search to take an ImageStack object instead of a StackSearch object. It now creates and sets the StackSearch object itself.


          Remove analysis_utils.py

b4334e6

jeremykubica requested a review from maxwest-uw

March 5, 2024 14:20

Contributor Author

jeremykubica commented Mar 5, 2024

Closes #507

maxwest-uw approved these changes

View reviewed changes

jeremykubica merged commit f39e379 into main

2 checks passed

jeremykubica deleted the analysis_utils branch

March 5, 2024 21:54

jeremykubica mentioned this pull request

Remove analysis_utils.py #507

Closed

DinoBektesevic mentioned this pull request

Add Logging module to the code. #524

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet