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

Segmentation: refactoring #25

Open
wants to merge 55 commits into
base: master
Choose a base branch
from
Open

Segmentation: refactoring #25

wants to merge 55 commits into from

Conversation

zmughal
Copy link
Member

@zmughal zmughal commented Mar 28, 2016

No description provided.

TODO: fix the FFT to treat data for ifftn as conjugate symmetric. This
will fix the code for finding the maximum response.
This allows for reusing a container.
- One function did not return the correct variable.
- Another called a function before the container length was set.
Wrap all the seed values inside a struct
@coveralls
Copy link

Coverage Status

Coverage increased (+2.4%) to 79.839% when pulling 2dff9bb on segmentation into 03d208f on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.4%) to 79.839% when pulling de503ea on segmentation into 03d208f on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+2.4%) to 79.839% when pulling f9c5cf6 on segmentation into 03d208f on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants