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

write Field II mesh interpolation function #9

Open
mlp6 opened this issue Aug 11, 2017 · 6 comments
Open

write Field II mesh interpolation function #9

mlp6 opened this issue Aug 11, 2017 · 6 comments
Assignees

Comments

@mlp6
Copy link
Owner

mlp6 commented Aug 11, 2017

instead of having to always re-run Field II when using a new mesh, we should have a function that will take an existing 3D Field II intensity output and 3D map it onto a new grid

@mlp6
Copy link
Owner Author

mlp6 commented Aug 11, 2017

@mlp6
Copy link
Owner Author

mlp6 commented Aug 11, 2017

This seems to be more important of a feature now for a few reasons:

  1. Field II doesn't play nicely with RAM and meshes that have large absolute extents (regardless of mesh density), which reared its head when trying to simulate the 4V1c
  2. field2dyna doesn't need to chug through tons of points when doing a shear wave simulation and the vast majority of the nodes are outside the footprint of the active array aperture
  3. Migrating to the C-version of Field II Pro would be easier to just specify a 3D array of points in the source code than to have to read in the nodes.dyn file.

@mlp6
Copy link
Owner Author

mlp6 commented Feb 4, 2018

@mjr52 your task to integrate the HFR-ARFI beams in falls under this Issue. Would be nice to make something that works for both applications (HFR-ARFI and the use-case detailed above).

@mlp6 mlp6 assigned mlp6 and unassigned mjr52 Feb 6, 2020
@mlp6
Copy link
Owner Author

mlp6 commented Aug 25, 2023

@jrichardson97 your branch that I'm merging in (#121) does this now, correct?

@jrichardson97
Copy link
Collaborator

Yes. I'll make an example with a sample load file and the interpolation constraints

@mlp6
Copy link
Owner Author

mlp6 commented Aug 25, 2023

@jrichardson97 you will need to create a new branch off of main for that as I just merged in your large pull request after some tweaks to the testing environment. I'll let you close this Issue when that PR is ready to go.

@mlp6 mlp6 assigned jrichardson97 and unassigned mlp6 Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants