Skip to content

feat: add support for keyword arguments in MATLAB interface - #132

Open
lmulder1991 wants to merge 8 commits into
mainfrom
feat--julia-kwargs-support
Open

feat: add support for keyword arguments in MATLAB interface#132
lmulder1991 wants to merge 8 commits into
mainfrom
feat--julia-kwargs-support

Conversation

@lmulder1991

@lmulder1991 lmulder1991 commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Added support for JULIA keyword arguments from MATLAB

Julia-like inline syntax ( 'name=value')
example: y = mjl.PkgName.FunctionName(args, kwargs)
Explicit kwargs object syntax ( 'jl.kwargs(...)')
Fixed signature validation edge cases
Enforced kwargs position rules: throw matfrostjulia:invalidKwargsPosition

@github-actions github-actions Bot added the minor Minor version bump (new features) label Aug 19, 2026
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown

Test Results

    4 files     32 suites   11m 49s ⏱️
4 078 tests 4 078 ✅ 0 💤 0 ❌
8 156 runs  8 156 ✅ 0 💤 0 ❌

Results for commit f097822.

♻️ This comment has been updated with latest results.

@codecov-commenter

codecov-commenter commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.58824% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.21%. Comparing base (62ac844) to head (f097822).

Files with missing lines Patch % Lines
src/matlab/@kwargs/kwargs.m 77.77% 4 Missing ⚠️
src/matlab/@matfrostjulia/matfrostjulia.m 94.02% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #132       +/-   ##
===========================================
+ Coverage   48.25%   64.21%   +15.96%     
===========================================
  Files           3        4        +1     
  Lines         143      204       +61     
===========================================
+ Hits           69      131       +62     
+ Misses         74       73        -1     

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

restore origin version 

Signed-off-by: lennard l Mulder <88686866+lmulder1991@users.noreply.github.com>
downgrade Ubuntu

Signed-off-by: lennard l Mulder <88686866+lmulder1991@users.noreply.github.com>
@lmulder1991
lmulder1991 marked this pull request as ready for review August 20, 2026 09:28
Signed-off-by: lennard l Mulder <88686866+lmulder1991@users.noreply.github.com>
lift compatibility constraint

Signed-off-by: lennard l Mulder <88686866+lmulder1991@users.noreply.github.com>
Comment thread test/MATFrostTest/Project.toml Outdated
[deps]
MATFrost = "406cae98-a0f7-4766-b83f-8510a556e0e7"

[compat]

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think this project would also need to be updated, when creating a new release. @jorisbelierasml

@lmulder1991
lmulder1991 enabled auto-merge (squash) August 20, 2026 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Minor version bump (new features)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants