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

Elliposid distance computer #22

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

tischi
Copy link

@tischi tischi commented Sep 24, 2021

@NicoKiaru
We need some functionality to measure the distance of points in the original image.
Related to this: https://forum.image.sc/t/compute-volume-and-voxel-sizes-after-transformation/57728/9
Still working on it, but please let me know if something like this could be added to your repo.

@NicoKiaru
Copy link
Member

I'm not sure it's a good idea to add it to this repo. I think there are already a lot of unrelated things in this repo. My goal is rather to split it (the QuPath part could go in spimdata extras), the registration part somewhere else, etc.

So why not, but you know what I think of this repo's organisation

@tischi
Copy link
Author

tischi commented Sep 28, 2021

For my users it would be very convenient if they could install all the Ellipsoid related stuff from one update site, which seems to be ABBA, is it? Thus, it would be very nice if we could host this distance computation code next to the code where the EllipsoidTransforms are built and applied.
We can think of a long term better strategy, but I feel in terms of update site and dependency management, putting it here would be the easiest for now. OK?

@NicoKiaru
Copy link
Member

I can add one of your repos to the update site

@tischi
Copy link
Author

tischi commented Sep 28, 2021

I can add one of your repos to the update site

I currently don't have something that would lend itself as a place for this. I could make a new one, but just for one function it's maybe a bit of an overkill?

@tischi
Copy link
Author

tischi commented Sep 29, 2021

@NicoKiaru Maybe the best is if I provide this functionality as a groovy script to my users? Then I can quickly change it and you don't need to host it. What do you think?

@NicoKiaru
Copy link
Member

We can put groovy scripts also in the update site.

What I was dreaming of would be to have github gists synchronized with Fiji.

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