search

kristinbranson / APT

Animal Part Tracker
GNU General Public License v3.0
72 stars 14 forks source link

APT doesn't save DL predictions in lbl file #290

Open arobie opened 5 years ago

arobie commented 5 years ago

Is this the final state? If so, we should warn user when they are closing a project if they have 'new' DL tracking predictions that they should export if they want to keep. I find it a bit confusing because 1. CPR stores tracking in project, so i would naively assume that DL predictions will be saved too, and 2. Projects remember that you've imported trk results and shows imported predictions that next time you load.

arobie commented 5 years ago

Also - it asks me to resave my project because 'new frames tracked since...' which if they aren't being saved in project is just confusing...

allenleetc commented 5 years ago

Good points, anyone/everyone please weigh in with thoughts

allenleetc commented 5 years ago

Maybe another option in between "don't save anything" and "always save everything" is to enable saving of selected tracking results to a project at user's discretion, eg "keep these tracking results in the project under label/tag \<somethingInformative>, similar to https://github.com/kristinbranson/APT/wiki/Multiple-Tracking-Results-Comparison. This would be in addition to trkfile export

mkabra commented 5 years ago

I think we don't need to give the user fine level control in this situation. So either save all or none would be fine.

On Wed, May 1, 2019 at 6:34 PM Allen Lee notifications@github.com wrote:

Maybe another option in between "don't save anything" and "always save everything" is to enable saving of user-specified tracking results to a project, eg "keep these tracking results in the project under label/tag , sort of like in https://github.com/kristinbranson/APT/wiki/Multiple-Tracking-Results-Comparison https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_kristinbranson_APT_wiki_Multiple-2DTracking-2DResults-2DComparison&d=DwMFaQ&c=LU6cRtx0xgB8s29tIz9Olw&r=Buc6iE8bkJgk8p2ZXUtrDhOgdAkeU00fkbaVIbsSBgI&m=CKReOXmoazrkuqRVXPYZE7KaSOGoBiW0JSNl8sj6wc4&s=8Th0rSs65CvJuNW_SfvznxO4HZKLINCWiM7askUR_CU&e=. This would be in addition to trkfile export

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_kristinbranson_APT_issues_290-23issuecomment-2D488464746&d=DwMFaQ&c=LU6cRtx0xgB8s29tIz9Olw&r=Buc6iE8bkJgk8p2ZXUtrDhOgdAkeU00fkbaVIbsSBgI&m=CKReOXmoazrkuqRVXPYZE7KaSOGoBiW0JSNl8sj6wc4&s=Zf27saPEUGPWc2aq9ByLCFC440zaEsT2J5hNvQcPCgQ&e=, or mute the thread https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AAJNKY3JYMVT6SO47UQVHNDPTILF7ANCNFSM4HIZ3SYQ&d=DwMFaQ&c=LU6cRtx0xgB8s29tIz9Olw&r=Buc6iE8bkJgk8p2ZXUtrDhOgdAkeU00fkbaVIbsSBgI&m=CKReOXmoazrkuqRVXPYZE7KaSOGoBiW0JSNl8sj6wc4&s=RvrvNGUnmB9-Of-vDa9Ymtfb9aywZvCfPHEU-rFEFDY&e= .