Skip to content

Bridge: sync authenticate, testLogin and profile to NDI-matlab 3cb19a85f (NDI-matlab#1049) - #342

Merged
stevevanhooser merged 1 commit into
mainfrom
claude/ci-failures-gyepvc
Oct 23, 2026
Merged

stevevanhooser merged 1 commit into
mainfrom
claude/ci-failures-gyepvc

Conversation

@stevevanhooser

Copy link
Copy Markdown
Contributor

Summary

VH-Lab/NDI-matlab#1049 made NDI_CLOUD_USERNAME / NDI_CLOUD_PASSWORD choose the account a new login uses, ahead of a MATLAB Vault entry for a different email. Setting them, or calling switchProfile, now actually switches account. In MATLAB, authenticate.m changed in code, and testLogin.m and profile.m changed only in their help text.

Python already had that order: an existing token, then the environment's credentials, then the saved profile. switch_profile already took effect the same way. So no code changes: the three bridge entries move to 3cb19a85f, each with a note that the two languages now agree. This clears their drift in test_matlab_bridge_hashes.

Still drifted, and not in this PR

export.all_binary, export.autoMultiplier and export.oneProbe (src/ndi/fun/probe) were changed by VH-Lab/NDI-matlab#1050:

  • autoMultiplier now treats whole-number doubles as raw counts;
  • all_binary chooses the multiplier per probe;
  • #1050 also adds a new jrclust.m exporter.

Those need a real port, so test_matlab_bridge_hashes stays red on those three until a follow-up lands.

Validation

  • The bridge tests (hashes, completeness, status, python paths, properties) were run against NDI-matlab main at 3cb19a85f: 198 passed, 1 failed. The failure is the probe-export drift above; the three entries changed here are no longer flagged.
  • This changes YAML only, so there's no Python code to lint or format.

Generated by Claude Code

VH-Lab/NDI-matlab#1049 made NDI_CLOUD_USERNAME / NDI_CLOUD_PASSWORD pick the
account a new login uses, ahead of a MATLAB Vault entry for a different
email, so setting them (or switchProfile) actually switches account.
authenticate.m changed; testLogin.m and profile.m only in their help text.

Python already had that order -- an existing token, then the environment's
credentials, then the saved profile -- and switch_profile already took effect
the same way, so no code changes: the three entries move to 3cb19a85f with a
note saying the two sides now agree. This clears their drift in
test_matlab_bridge_hashes.

Still drifted, and left for a separate port: export.all_binary,
export.autoMultiplier and export.oneProbe, moved by VH-Lab/NDI-matlab#1050.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KChjwyZT9314dmEuS5ovUP
@stevevanhooser
stevevanhooser merged commit ce02c25 into main Oct 23, 2026
14 of 15 checks passed
@stevevanhooser
stevevanhooser deleted the claude/ci-failures-gyepvc branch October 23, 2026 21:06
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.

2 participants