-
Notifications
You must be signed in to change notification settings - Fork 12
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
Random sampling #220
Random sampling #220
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great idea for an additional tool. A bunch of comments mostly about aligning this function with other similar tools and also avoiding redundancy with PR #206
@Daval-G What do you think of this updated version ? also do not hesitate to check the example as well |
2ece4be
to
c4dc349
Compare
c4dc349
to
630486c
Compare
@paquiteau I am starting again to look at this request, but I won't finish tonight. I see that a bunch of my old comments are un-answered, could you have a look ? Thx |
I think now I have address everything you asked for :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great improvements, only small comments remaining. Would you like to give a try to type hints ? If not it's OK, it was an itch from me but I don't want to impose it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR adds random sampling extension for trajectories.
In particular, it adds a trajectory expansion function (
stack_random
) to stack 2D trajectory following a certain distribution.([ ] Add ND sampling (e.g poisson disc))An example of stack of spiral with uniform sampling (but fully sampled in the center of k-space)