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

angular_spectrum: implementation & test #83

Merged
merged 7 commits into from
Dec 28, 2022

Conversation

faridyagubbayli
Copy link
Collaborator

Adds implementation of the angular_spectrum function along with its test.

Additionally:

  • Moved TestRecordReader and check_kgrid_equality to a separate files under python_testers/utils
  • Fixed a bug in the next_pow2 method

Copy link
Owner

@waltsims waltsims left a comment

Choose a reason for hiding this comment

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

left one comment.

@faridyagubbayli faridyagubbayli merged commit 94b11db into master Dec 28, 2022
@faridyagubbayli faridyagubbayli deleted the testing/angularSpectrum branch December 28, 2022 20:47
waltsims pushed a commit that referenced this pull request Dec 29, 2022
waltsims added a commit that referenced this pull request Feb 2, 2023
* Add tone_burst collector

* remove all import-alls

* remove import all form example

* Update test_cpp_io_in_parts.py

change file names

* angular_spectrum: implementation & test (#83)

* Add collector and test for gradientSpect

* Add collector and test for gaussian

* Move gaussian test.

* test tenenbaum sharpness

* split sharpness tests

* Refactor filter tests

* test smooth

* Add hounsfield2density collector and test

* Add add CWSignals collector and test

* Extend collector for resize.

* skip nearest neighbor test case for resize

* Add timing again

* Add spaces to EOFs

* Add __len__ to RecordReader

* Test resize with RecordRecorder

* Add TODO

* Use TestRecorder for tone_burst

* Test and pass signal shorter than offset

---------

Co-authored-by: Walter Simson <[email protected]>
Co-authored-by: Farid Yagubbayli <[email protected]>
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.

None yet

2 participants