kristinbranson / APT

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

Macros in Batch/External Tracking UI #346

Open allenleetc opened 3 years ago

allenleetc commented 3 years ago

Enhancements for macro functionality used to specify trkfile outputs in Track>Track Multiple Videos UI:

Split issue out from #344.

arobie commented 3 years ago

Editbox for "Define trkfile using macros" can be pre-populated with default export trkfile which is currently $movdir/$movfile$projfile$trackertype

This doesn't work for me on develop? Am I missing something? I still find this very hard to use without default trk naming as a starting point. Can you tell me an expected work flow? I tried pasting the above default macro into the GUI, and then selecting a movie using the selection gui. I still had to provide a handmade trk in the file selection gui to be able to return the main GUI. If I hit apply next to the macro, then it changes my trk file path to the macro, but it doesn't display the trk that will be used. I don't know if this is correct or not. If I then try to use file selection gui again, it shows the macro, but won't let me return to the main gui because there's an error about incorrect or missing data in the trk field.

allenleetc commented 3 years ago

Yes you are right, hand-entering macros into the "Define trk using macros" box doesn't work for me either. The issue description implies that it would/might.

I don't remember exactly what I tested in terms of macro-functionality on the first pass, but it wasn't much -- basically this issue is a placeholder for all macro-related functionality in that UI. I'll make another pass on this.

What's the most important core issue here? I am guessing the main pain is clicking or typing a full path to the output trkfile, which for your workflow is a deeply nested file and eg adjacent to the movie-to-be-tracked. If so, I'm not sure we need the "Define trk using macros" field in the "outer" UI? We could just have the "inner"/detail UI be prepopulated with the default macro (and have that macro work) and we would be good?

arobie commented 3 years ago

Yes, if I'm reading correctly, prepopulating with the default trk in the inner GUI would be perfect for me!

If you want to be more general: Maybe instead of the macros in the outer field you could just show the default trk file name and people could edit that if they wanted, and whatever was in that field would auto populate inner trk full path with movie path?

Have you seen users organizing their data in the 'other' way? With unique movie names? How does auto trk work for that work flow?

roian commented 3 years ago

ran into this myself. i would prefer a pre-populated trk file named based on the movie file, or some way to know a) what macros are available and b) what their current values are.

also, small GUI issue: the "add movie" button shows below the interactive gui for adding a specific movie, and i keep clicking it instead of the "done" button. either grey out while adding a movie, or have the sub-window cover it?

allenleetc commented 3 years ago