-
Notifications
You must be signed in to change notification settings - Fork 15
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
How to convert ROI coordinates from relative to pixels #232
Comments
Thanks for your note. I agree, it's pretty cumbersome to use ROIs straight from the file metadata 😂 |
Dear Talley,
Thanks for your response.
I think you are more fluent in github than me, so please do the PR.
The biggest challenge was that f.shape array has the dimensions in an unexpected order (height,width).
I think the interface should support moving ROIs as well.
I expect that the animsParam array has more entries in that case (one for every control frame ?)
WR,
Kees
Dr. Kees van der Oord
European Support Specialist
T: +31207099246
M: +31623367589
E: ***@***.***
Website
Nikon Europe B.V.
Stroombaan 14
1181 VX Amstelveen
Netherlands
Registered Chamber of Commerce Amsterdam No. 34139593
VAT No. NL 8091.99.889.B.01
This e-mail may contain information that is PRIVILEGED and/or CONFIDENTIAL. This e-mail is intended solely for the individual or entity to whom it is addressed. If you are not, or have reason to believe you are not the intended recipient of this e-mail please contact us by reply e-mail as soon as possible. Immediately after contacting us, please destroy this e-mail (including attachment(s)) from your computer system(s). Any unauthorized copying, disclosure or distribution of this e-mail (including attachment(s)) is not allowed. Nikon Europe BV hereby disclaims any and all liabilities which may arise in relation to any of its electronic communication. No obligation is entered into by virtue of this e-mail, unless confirmed in writing and duly signed by an authorized representative of Nikon Europe BV
From: Talley Lambert ***@***.***>
Sent: 20 June, 2024 16:29
To: tlambert03/nd2 ***@***.***>
Cc: Oord, Kees van der ***@***.***>; Mention ***@***.***>
Subject: Re: [tlambert03/nd2] How to convert ROI coordinates from relative to pixels (Issue #232)
hey @Kees-van-der-Oord-Nikon<https://github.com/Kees-van-der-Oord-Nikon>
Thanks for your note. I agree, it's pretty cumbersome to use ROIs straight from the file metadata 😂
This is great and i'd be happy to add a convenience like this to the code base. Would you be interested in opening a PR to that effect? (Happy to do it myself if not)
—
Reply to this email directly, view it on GitHub<#232 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AXSSBOSB2W2GBE4FDVE7EPTZILRJZAVCNFSM6AAAAABJUA6XNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBQHA2TGNJYGA>.
You are receiving this because you were mentioned.Message ID: ***@***.******@***.***>>
|
no worries, will do
yes (height, width) (row major) is the standard convention for arrays in python, numpy, etc.., so |
Description
This is not really an issue, just a suggestion to add to the documentation
What I Did
The code below converts the ROI coordinates from relative fractions to pixel coordinates.
The Ellipse ROI width and height are before rotation over the indicated angle.
The only missing detail is the inner circle of the Ring ROI.
WR,
Kees
The text was updated successfully, but these errors were encountered: