Skip to content
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

implement a dialog for importing externally generated tracks from csv #109

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

AnniekStok
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@cmalinmayor cmalinmayor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is mostly stuff we discussed Monday. I updated the file structure a bit to make an import_export folder, and I didn't see any circular import issues when I ran it, but let me know if there is anything weird. I'll keep thinking about how we want to generally handle attributes in a more consistent way too.

src/motile_plugin/import_export/export_solution_to_csv.py Outdated Show resolved Hide resolved
src/motile_plugin/import_export/load_tracks.py Outdated Show resolved Hide resolved
src/motile_plugin/import_export/load_tracks.py Outdated Show resolved Hide resolved
if segmentation is not None:
segmentation = np.expand_dims(
segmentation, axis=1
) # do we need to keep the empty hypothesis dim?
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, for now! Until we update to motile_toolbox v0.4.0

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! Thanks for your comments! Yes we need to update it again when we get rid of the hypothesis segmentation and make the node_ids unique. I also included a basic test checking if the segmentation pixel value at node coordinates is in agreement with the provided seg_id in the csv file.

AnniekStok and others added 18 commits November 29, 2024 08:51
…uess fields and updates the name of the custom attributes when a column is selected
…_tracks instead. Update also view_external_tracks.py and documentation
…dinates does not match with the provided seg_id from the csv file, as this likely indicates that the wrong pair of files has been selected. Remove the warning about viewing tracks that are not a motile run
Update conda config and pyproject napari entrypoint to tracker
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants