-
Notifications
You must be signed in to change notification settings - Fork 2
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
Stitched ROIS from IFCB1 and IFCB5 do not display correctly #95
Comments
@eepeacock I suspect this is a dashboard issue not an ifcb-annotate issue. Do the ROIs look the same in ifcb-annotate and ifcb-dashboard? Is this happening for all stitched ROIs or only some? Can you post a few examples here? |
But if you right click to link in the dashboard you get: |
Hmm...what did you mean by "it takes up the size and dimension of the stitched pair"? That looks like just the first ROI. |
I think it's just the single ROI. We'll have to ask Joe what api call is being made from ifcb-annotate to ifcb-dashboard--and why that is returning unstitched ROIs. Does the bin zip link for this bin provide the stitched ROIs or the separate singles? |
THe images are definitely distorted. It makes it easy to spot them. I would not have noticed the issue if I was just viewing one ROI.... it would just look like one ROI. Here is another exampled: https://ifcb-data.whoi.edu/mvco/IFCB1_2007_155_234217_02186.html |
There's an interaction between the dashboard and annotate that fetches ROI metrics like width/height independently of the zip, to decide how to lay out the images before the zip is loaded. If there are discrepancies between the image metrics returned by the dashboard and the zip returned by the dashboard, it would likely cause this problem. I don't think I need more examples @eepeacock , I will investigate the likely cause. Thanks |
Thanks! |
I believe this is the line where I don't stitch when returning roi sizes to annotate; should be a quick fix |
Looks like the same issue in the production of the zip. I will confirm and it'll require fixes in both repos if my diagnosis is correct. |
Oh, man. I just realized that this bug in the bin zips (stitched ROIs not there) cascades through the complete v4 production of blobs and features.... [We did not have this problem when v2 blobs and features were created.] |
Stitched ROIs appear as:
First ROI of pair only is displayed, but it takes up the size and dimensions of the stitched pair.
Since new CNN results are available and I am reviewing some older data, this is a problem.
The text was updated successfully, but these errors were encountered: