Conversation
…izedRayTracing
…lly and no output argument was requested
fix that optimization status window vanishes, in newer matlab version, and that DVH stats table is displayed incorrectly
Updates to GUI
…ves into structs and correctly showing robustness when the instance is stored in the cst
…to feature/single_precision_dosecalc # Conflicts: # matRad/rayTracing/matRad_rayTracing.m
…atives) for fmincon
…set to true. Also reset it to false by default.
Single precision computations and rudimentary GPU support
Test Results 3 files ± 0 3 suites ±0 31m 3s ⏱️ - 13m 35s Results for commit 2da1de6. ± Comparison against base commit 162bde8. This pull request removes 8 and adds 46 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
# Conflicts: # .miss_hit # matRad/doseCalc/+DoseEngines/@matRad_ParticleFREDEngine/calcDose.m # test/doseCalc/test_FREDEngine.m
…n favor of no-index in sphinx 8
…d into docs/wiki_conversion # Conflicts: # docs/cite.rst # docs/conf.py # docs/modules.rst
…ocs/wiki_conversion
…inx, also apply reformatting
…d errors from sphinx build
# Conflicts: # matRad/matRad_version.m
…regarding the logger in >=9
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
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.
Patch Release 3.2.1
pln.propDoseCalc.precisionfor using single precision in dose calculation (single precision dij storage in >=R2025a). Standard is stillpln.propDoseCalc.precision = 'double', but this will be changed to 'single'in the next major release.pln.propDoseCalc.traceOnDoseGridfor performing rayTracing on dose grid in dose calculation. Standard is stillfalse, but this might change in the next major release after some accuracy benchmarking.pln.propOpt.enableGPUfor using GPU throughout optimization (only within function evaluations), and supplied some helper functions to copy matRad's data structures. Disabled by default.Release Checklist
Wiki adaptedSphinx-build succeedsCopilot Summary:
This pull request introduces several improvements to code quality, consistency, and maintainability. The main changes include adding pre-commit hooks and code style enforcement, updating several example scripts for clarity and correctness, and making minor documentation and spelling corrections.
Code Quality and Tooling Improvements:
.pre-commit-config.yamlfile to enable pre-commit hooks for code formatting, YAML checks, spell checking, and integration with the MISS_HIT linter and metrics for MATLAB files..miss_hitconfiguration file to enforce consistent code style, naming conventions, and complexity metrics for MATLAB code using the MISS_HIT tool..codespellrcfile to configure codespell and ignore specific false positives.CI/CD Pipeline Enhancements:
coverage-report.yml) to setcontinue-on-error: truefor both Coveralls and Codecov upload steps, ensuring that coverage upload failures do not fail the entire workflow.Example Script Updates:
matRad_example1_phantom.m,matRad_example2_photons.m,matRad_example3_photonsDAO.m,matRad_example4_photonsMC.m,matRad_example5_protons.m,matRad_example6_protonsNoise.m,matRad_example10_4DphotonRobust.m,matRad_example11_helium.m,matRad_example12_simpleParticleMonteCarlo.m,matRad_example14_spotRemoval.m,matRad_example16_photonMC_MLC.m,matRad_example17_biologicalModels.m,matRad_example18_FREDMC.m,matRad_example19_CT_sCT_DVH_difference_photons.m,matRad_example20_VHEE.m) to remove redundant calculation and assignment ofnumOfBeamsand to simplify the assignment ofisoCenter. Now,isoCenteris directly assigned usingmatRad_getIsoCenter, improving clarity and reducing errors. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18]Documentation and Spelling Corrections:
matRad_example15_brachy.m(e.g., "frequantly" → "frequently", "eather" → "either", corrected method name casing, improved figure formatting, and other minor comment corrections). [1] [2] [3] [4] [5] [6] [7]These changes collectively improve the maintainability, reliability, and clarity of the codebase.