Skip to content

Conversation

@talg2324
Copy link
Contributor

@talg2324 talg2324 commented Mar 6, 2024

No description provided.

@codecov
Copy link

codecov bot commented Mar 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.77%. Comparing base (fbef132) to head (2f8c853).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #341   +/-   ##
=======================================
  Coverage   67.77%   67.77%           
=======================================
  Files          48       48           
  Lines        7068     7068           
  Branches     1605     1605           
=======================================
  Hits         4790     4790           
  Misses       1697     1697           
  Partials      581      581           
Flag Coverage Δ
3.10 67.92% <ø> (ø)
3.11 67.92% <ø> (ø)
3.12 67.92% <ø> (ø)
3.9 67.73% <ø> (ø)
ubuntu-latest 67.71% <ø> (ø)
windows-latest 67.71% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@djps djps self-requested a review March 6, 2024 21:59
djps
djps previously requested changes Mar 12, 2024
([original example](http://www.k-wave.org/documentation/example_at_linear_array_transducer.php), [![Open In Colab](https://colab.research.google.com/assets/colab-badge.svg)](https://colab.research.google.com/github/waltsims/k-wave-python/blob/master/examples/at_linear_array_transducer/at_linear_array_transducer.ipynb))
- [Photoacoustic Waveforms](ivp_photoacoustic_waveforms/) ([original example](http://www.k-wave.org/documentation/example_ivp_photoacoustic_waveforms.php), [![Open In Colab](https://colab.research.google.com/assets/colab-badge.svg)](https://colab.research.google.com/github/waltsims/k-wave-python/blob/master/examples/ivp_photoacoustic_waveforms/ivp_photoacoustic_waveforms.ipynb))
- [Controling the PML](na_controlling_the_pml/)
- [Controlling the PML](na_controlling_the_pml/)
Copy link
Owner

Choose a reason for hiding this comment

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

Thanks for catching these typos.

Comment on lines +117 to 119
steering_angle_max: max steering angle [deg]
steering_angle: steering angle [deg]
Copy link
Owner

Choose a reason for hiding this comment

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

nice.

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.

thanks for the PR

@waltsims waltsims linked an issue Mar 16, 2024 that may be closed by this pull request
@waltsims
Copy link
Owner

waltsims commented Mar 19, 2024

Steps to merge:

  • check symmetry of visualization @djps

@talg2324 talg2324 requested a review from djps March 22, 2024 16:27
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.

looks good now.

@waltsims waltsims dismissed djps’s stale review March 22, 2024 22:01

issues addressed

@waltsims waltsims merged commit 7d25625 into waltsims:master Mar 22, 2024
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.

[Example] us_bmode_phased_array

3 participants