Bump openlifu to v0.20.0#563
Merged
Merged
Conversation
Notable openlifu changes coming in with this bump: - Photoscan reconstruction updated for Meshroom 2025 compatibility (end users will need Meshroom 2025 rather than 2023.3). - Transducer/Element: `sensitivity` can now be scalar or a frequency-dependent list of (frequency, value) tuples; new `crosstalk_frac` / `crosstalk_dist` fields on Transducer; removed `impulse_response` / `impulse_dt` (serialized back-compat preserved via `from_dict`). - `Protocol.calc_solution` refactored: simulation logic moved into new `Solution.simulate` method; dropped `use_gpu` kwarg in favor of `_force_cpu`; added optional `params` kwarg. - `SolutionAnalysis` gains `focal_gain` and `estimated_tx_temperature_rise_C` fields (plus a `model_tx_temperature_rise` utility). - `Database`: removed the unused `systems/` subdirectory and all `*_system*` methods (`write_system`, `load_system`, `get_system_ids`, etc.). - `TransducerArray.get_concave_cylinder` parameter handling generalized to accept heterogeneous per-module transducers and more flexible gap/dth/roc combinations. - LIFU firmware/IO updates (DFU support, user config, OW_CMD->OW_CTRL rename, bulk TX read) — `LIFUInterface` public API unchanged. - New `xdc.util` helpers for reading transducer test reports and building sensitivity/focal-gain matrices.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This version bump will be followed by a SlicerOpenLIFU release v1.19.0, and then a custom app release v1.12.0. Release notes we can use:
Notable openlifu changes coming in with this bump
sensitivitycan now be scalar or a frequency-dependent list of (frequency, value) tuples; newcrosstalk_frac/crosstalk_distfields on Transducer; removedimpulse_response/impulse_dt(serialized back-compat preserved viafrom_dict).Protocol.calc_solutionrefactored: simulation logic moved into newSolution.simulatemethod; droppeduse_gpukwarg in favor of_force_cpu; added optionalparamskwarg.SolutionAnalysisgainsfocal_gainandestimated_tx_temperature_rise_Cfields (plus amodel_tx_temperature_riseutility).Database: removed the unusedsystems/subdirectory and all*_system*methods (write_system,load_system,get_system_ids, etc.).TransducerArray.get_concave_cylinderparameter handling generalized to accept heterogeneous per-module transducers and more flexible gap/dth/roc combinations.LIFUInterfacepublic API unchanged.xdc.utilhelpers for reading transducer test reports and building sensitivity/focal-gain matrices.Not much review needed; I ran our test suite and everything passed. I tried a small workflow and it's all good.