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.

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

ENH: Progress reporting for antsAI #1389

Merged
merged 3 commits into from
Jul 12, 2022
Merged

ENH: Progress reporting for antsAI #1389

merged 3 commits into from
Jul 12, 2022

Conversation

cookpa
Copy link
Member

@cookpa cookpa commented Jul 11, 2022

Proof of concept for progress reporting in antsAI. Currently prints every iteration, which is potentially a lot - could be smarter and print progress at intervals.

Fixes #439

@cookpa cookpa marked this pull request as ready for review July 12, 2022 15:58
@cookpa
Copy link
Member Author

cookpa commented Jul 12, 2022

Updated with some improvements. Outputs to terminal for every [(total iterations) / 10] iterations, and reports the best metric value at the end.

@ntustison
Copy link
Member

Thanks for this @cookpa

@cookpa cookpa merged commit 11284b7 into master Jul 12, 2022
@cookpa cookpa deleted the antsAIObserver branch July 12, 2022 17:05
@akvilonBrown
Copy link

akvilonBrown commented Apr 26, 2023

Hello.
I'm trying to apply ANTS for pairwise registration. My objects are 3D NMR images of plant seeds, they can have different orientations, and I understood that antsRegistration is not supposed to work with drastic affine differences.
So I tried antsAI to produce a rough affine transformation, but it seems it doesn't work as expected.
I created a pair to test - the second object is rotated 60 deg on one of the 3D axes.
antsRegistration can manage to find a rigid transformation (for greater rotations - it doesn't find), but antsAI gives a weird transformation.
When I apply antsApplyTransforms, the rotation is wrong. (It is ok with matrix from antsRegistration when rotation is small.)
I don't know how to debug and what am I missing.
May I create an issue (I can share data and commands) or maybe you can suggest how to track the process in antsAI ?

@cookpa
Copy link
Member Author

cookpa commented Apr 26, 2023

Yes please open an issue following the issue template, with a reproducible example (ie, something we can download and run without modification)

@akvilonBrown
Copy link

Thank you!
#1529

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.

Feature Request: more verbosity for antsAI
3 participants