Major changes and updates to facemapSVD pipeline

[ttngu207]

• Rename SVD component-related table attributes
• Update ingestion to handle FullSVD analysis
• Remove redundant helper functions

[MilagrosMarin]

I would suggest to make the function private and rename it to _run_facemap_process().

[MilagrosMarin]

An example here, _run_sorter() in element-array-ephys.

[MilagrosMarin]

Following other elements, such as element-deeplabcut setup, I would move the installation of the GUI to extras_require and make it optional.

[kushalbakshi]

Looks good to me. If you could update the run_facemap_process function based on Milagros' comment above, it will ensure consistency in how we deploy the memoized_result functionality across Elements. Otherwise, this is ready to merge.

[ttngu207]

Looks good to me. If you could update the run_facemap_process function based on Milagros' comment above, it will ensure consistency in how we deploy the memoized_result functionality across Elements. Otherwise, this is ready to merge.

@MilagrosMarin @kushalbakshi new commit added to address your comment.
There's one inconsistency in this soft convention we have here, in element-deeplabcut, we name the function do_analyze_videos (not _do_analyze_videos)
I'm fine either way, this is a minor point, it doesn't affect any functionality - it's good to stay consistent

[kushalbakshi]

Looks good to me. If you could update the run_facemap_process function based on Milagros' comment above, it will ensure consistency in how we deploy the memoized_result functionality across Elements. Otherwise, this is ready to merge.

@MilagrosMarin @kushalbakshi new commit added to address your comment. There's one inconsistency in this soft convention we have here, in element-deeplabcut, we name the function do_analyze_videos (not _do_analyze_videos) I'm fine either way, this is a minor point, it doesn't affect any functionality - it's good to stay consistent

Yes, it's probably a good idea to update that one too. As a general rule, any function within the make should probably follow the private function convention just to discourage its use outside the make