Skip to content

Use current recording stop API internally - #640

Merged
michalharakal merged 1 commit into
developfrom
feature/recording-api-cleanup-pr5
May 28, 2026
Merged

michalharakal merged 1 commit into
developfrom
feature/recording-api-cleanup-pr5

Conversation

@michalharakal

Copy link
Copy Markdown
Contributor

Closes #639. Summary: replaces internal stopRecordingAndGet() usage with stopRecording(), which already returns the tape; removes an unnecessary non-null assertion in the eager-vs-recording test. Validation: ./gradlew :skainet-compile:skainet-compile-dag:jvmTest --no-daemon --stacktrace; ./gradlew :skainet-compile:skainet-compile-dag:compileTestKotlinIosArm64 --no-daemon --stacktrace.

Replace internal uses of stopRecordingAndGet() with stopRecording(), which already returns the tape, and remove the unnecessary test non-null assertion.

Refs #639
@michalharakal
michalharakal merged commit 7a0d223 into develop May 28, 2026
6 checks passed
@michalharakal
michalharakal deleted the feature/recording-api-cleanup-pr5 branch May 28, 2026 14:56
MacOS pushed a commit to MacOS/SKaiNET that referenced this pull request Jul 10, 2026
Bumps VERSION_NAME 0.25.0 → 0.26.0 with the matching CHANGELOG entry,
README "What's New in 0.26.0" section, and sample-dependency version
bumps across the how-to and tutorial pages.

The 0.26.0 release rolls up the first-class tanh activation primitive
(SKaiNET-developers#630/SKaiNET-developers#631), the CPU tensor convert op (SKaiNET-developers#636), and test/build/CI
hygiene (SKaiNET-developers#633, SKaiNET-developers#634, SKaiNET-developers#638, SKaiNET-developers#640, SKaiNET-developers#645).

Operator docs regenerated so the version stamp reads 0.26.0. The
native-FFM plan draft was removed from the repo root; a short design
summary now lives in reference/architecture.adoc.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

Remove deprecated recording helper usage in DAG module

1 participant