rays-video 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ea13fca90e2e0f9c175fad5283bb6f0cc8c763caf6386693151c9f8d477a1c1
-  data.tar.gz: 4bfa9890019ed3d63c14272da0c4a1abbad784881c96b5b9078a60e4c34ee826
+  metadata.gz: 2bfe8646da57777c45c2061d2343f5872f095c5b05166abe3264b6c47150756b
+  data.tar.gz: b7216425b5f1941be61dfe3aa7744cedcfa72e21a95e552e466e28a3dfe623e8
 SHA512:
-  metadata.gz: 3449d2b2ed3cc417529de8639d4be7438af541881b7e7e736ae79e096e8f92c4c0b235ab540f2759012a8adad9087533f6486c251f4feeedcfafc08cdcf36b4a
-  data.tar.gz: 72bbfb7b9d3c7c84e41f89bc9d8cbf7bb70ca1c55b2438a296f56468de5752af0700c302715216ffad050aa73234317a51e70e26cc9a1dbbc0ab89cc42bf7743
+  metadata.gz: 120b483fdbe17c7890739cf20e2f6f143d0a47e962789aa76f49b8d118fe6f9f34f047a9370c0bd53d870060704c962606ae9c489ce6432cf1c8471483e183e9
+  data.tar.gz: 318a8c0ffb1c3f95e0dd1946863fbc6b768ddad518a1f9b3727c6f629f19174eb0ca47e724ed7a0fbb7deda6947644991aaf902a823785c4f09e4d16e958958c

data/.doc/ext/rays-video/video.cpp

@@ -9,7 +9,7 @@ RUCY_DEFINE_VALUE_FROM_TO(RAYS_VIDEO_EXPORT, Rays::Video)
 
 #define THIS  to<Rays::Video*>(self)
 
-#define CHECK RUCY_CHECK_OBJECT(Rays::Video, self)
+#define CHECK RUCY_CHECK_OBJ(Rays::Video, self)
 
 
 static
@@ -21,7 +21,7 @@ VALUE alloc(VALUE klass)
 static
 VALUE initialize(VALUE self, VALUE width, VALUE height, VALUE fps, VALUE pixel_density)
 {
-	RUCY_CHECK_OBJ(Rays::Video, self);
+	RUCY_CHECK_PTR(Rays::Video, self);
 
 	float fps_ = to<float>(fps);
 	*THIS = Rays::Video(
@@ -34,32 +34,32 @@ VALUE initialize(VALUE self, VALUE width, VALUE height, VALUE fps, VALUE pixel_d
 static
 VALUE initialize_copy(VALUE self, VALUE obj)
 {
-	RUCY_CHECK_OBJ(Rays::Video, self);
+	RUCY_CHECK_PTR(Rays::Video, self);
 
 	*THIS = to<Rays::Video&>(obj).dup();
 }
 
 static
-VALUE insert(VALUE self, VALUE index, VALUE image)
+VALUE play(VALUE self)
 {
 	CHECK;
-	THIS->insert(to<size_t>(index), to<const Rays::Image&>(image));
+	THIS->play();
 	return self;
 }
 
 static
-VALUE append(VALUE self, VALUE image)
+VALUE pause(VALUE self)
 {
 	CHECK;
-	THIS->append(to<const Rays::Image&>(image));
+	THIS->pause();
 	return self;
 }
 
 static
-VALUE remove(VALUE self, VALUE index)
+VALUE stop(VALUE self)
 {
 	CHECK;
-	THIS->remove(to<size_t>(index));
+	THIS->stop();
 	return self;
 }
 
@@ -85,6 +85,13 @@ VALUE height(VALUE self)
 	return value(THIS->height());
 }
 
+static
+VALUE pixel_density(VALUE self)
+{
+	CHECK;
+	return value(THIS->pixel_density());
+}
+
 static
 VALUE fps(VALUE self)
 {
@@ -93,102 +100,103 @@ VALUE fps(VALUE self)
 }
 
 static
-VALUE pixel_density(VALUE self)
+VALUE set_position(VALUE self, VALUE position)
 {
 	CHECK;
-	return value(THIS->pixel_density());
+	THIS->set_position(to<size_t>(position));
+	return position;
 }
 
 static
-VALUE size(VALUE self)
+VALUE get_position(VALUE self)
 {
 	CHECK;
-	return value(THIS->size());
+	return value(THIS->position());
 }
 
 static
-VALUE empty(VALUE self)
+VALUE set_time_scale(VALUE self, VALUE scale)
 {
 	CHECK;
-	return value(THIS->empty());
+	THIS->set_time_scale(to<float>(scale));
+	return scale;
 }
 
 static
-VALUE set_position(VALUE self, VALUE position)
+VALUE get_time_scale(VALUE self)
 {
 	CHECK;
-	THIS->set_position(to<size_t>(position));
-	return position;
+	return value(THIS->time_scale());
 }
 
 static
-VALUE get_position(VALUE self)
+VALUE insert(VALUE self, VALUE index, VALUE image)
 {
 	CHECK;
-	return value(THIS->position());
+	THIS->insert(to<size_t>(index), to<const Rays::Image&>(image));
+	return self;
 }
 
 static
-VALUE each(VALUE self)
+VALUE append(VALUE self, VALUE image)
 {
 	CHECK;
-	Value ret;
-	for (auto it = THIS->begin(), end = THIS->end(); it != end; ++it)
-		ret = yield(value(*it));
-	return ret;
+	THIS->append(to<const Rays::Image&>(image));
+	return self;
 }
 
 static
-VALUE to_image(VALUE self)
+VALUE remove(VALUE self, VALUE index)
 {
 	CHECK;
-	return value((Rays::Image) *THIS);
+	THIS->remove(to<size_t>(index));
+	return self;
 }
 
 static
-VALUE at(VALUE self, VALUE index)
+VALUE set_at(VALUE self, VALUE index, VALUE image)
 {
 	CHECK;
-	return value((*THIS)[(size_t) to<int>(index)]);
+	THIS->set(to<size_t>(index), to<const Rays::Image&>(image));
+	return image;
 }
 
 static
-VALUE play(VALUE self)
+VALUE get_at(VALUE self, VALUE index)
 {
 	CHECK;
-	THIS->play();
-	return self;
+	return value(THIS->get(to<size_t>(index)));
 }
 
 static
-VALUE pause(VALUE self)
+VALUE size(VALUE self)
 {
 	CHECK;
-	THIS->pause();
-	return self;
+	return value(THIS->size());
 }
 
 static
-VALUE stop(VALUE self)
+VALUE empty(VALUE self)
 {
 	CHECK;
-	THIS->stop();
-	return self;
+	return value(THIS->empty());
 }
 
 static
-VALUE set_time_scale(VALUE self, VALUE scale)
+VALUE each(VALUE self)
 {
 	CHECK;
-	THIS->set_time_scale(to<float>(scale));
-	return scale;
+	Value ret;
+	for (auto it = THIS->begin(), end = THIS->end(); it != end; ++it)
+		ret = yield(value(*it));
+	return ret;
 }
 
 static
-VALUE get_time_scale(VALUE self)
+VALUE to_image(VALUE self)
 {
 	CHECK;
-	return value(THIS->time_scale());
+	return value((Rays::Image) *THIS);
 }
 
 static
@@ -200,10 +208,12 @@ VALUE load(VALUE self, VALUE path)
 static
 VALUE exts(VALUE self)
 {
-	std::vector<Value> list;
-	for (const auto& ext : Rays::get_video_exts())
-		list.emplace_back(ext.c_str());
-	return array(list.data(), list.size());
+	const auto& exts = Rays::get_video_exts();
+
+	Value list = array(NULL, exts.size());
+	for (const auto& ext : exts)
+		list.push(ext.c_str());
+	return list;
 }
 
 
@@ -218,26 +228,27 @@ Init_rays_video ()
 	rb_define_alloc_func(cVideo, alloc);
 	cVideo.define_private_method("initialize!",     initialize);
 	rb_define_private_method(cVideo, "initialize_copy", RUBY_METHOD_FUNC(initialize_copy), 1);
-	cVideo.define_method("insert!", insert);
-	cVideo.define_method("append!", append);
-	cVideo.define_method("remove!", remove);
+	rb_define_method(cVideo, "play", RUBY_METHOD_FUNC(play), 0);
+	rb_define_method(cVideo, "pause", RUBY_METHOD_FUNC(pause), 0);
+	rb_define_method(cVideo, "stop", RUBY_METHOD_FUNC(stop), 0);
 	rb_define_method(cVideo, "save", RUBY_METHOD_FUNC(save), 1);
 	rb_define_method(cVideo, "width", RUBY_METHOD_FUNC(width), 0);
 	rb_define_method(cVideo, "height", RUBY_METHOD_FUNC(height), 0);
-	rb_define_method(cVideo, "fps", RUBY_METHOD_FUNC(fps), 0);
 	rb_define_method(cVideo, "pixel_density", RUBY_METHOD_FUNC(pixel_density), 0);
-	rb_define_method(cVideo, "size", RUBY_METHOD_FUNC(size), 0);
-	cVideo.define_method("empty?",        empty);
+	rb_define_method(cVideo, "fps", RUBY_METHOD_FUNC(fps), 0);
 	rb_define_method(cVideo, "position=", RUBY_METHOD_FUNC(set_position), 1);
 	rb_define_method(cVideo, "position", RUBY_METHOD_FUNC(get_position), 0);
-	rb_define_method(cVideo, "play", RUBY_METHOD_FUNC(play), 0);
-	rb_define_method(cVideo, "pause", RUBY_METHOD_FUNC(pause), 0);
-	rb_define_method(cVideo, "stop", RUBY_METHOD_FUNC(stop), 0);
 	rb_define_method(cVideo, "time_scale=", RUBY_METHOD_FUNC(set_time_scale), 1);
 	rb_define_method(cVideo, "time_scale", RUBY_METHOD_FUNC(get_time_scale), 0);
+	cVideo.define_method("insert!", insert);
+	cVideo.define_method("append!", append);
+	cVideo.define_method("remove!", remove);
+	cVideo.define_method("[]=",     set_at);
+	cVideo.define_method("[]",      get_at);
+	rb_define_method(cVideo, "size", RUBY_METHOD_FUNC(size), 0);
+	cVideo.define_method("empty?",  empty);
 	cVideo.define_method("each!", each);
 	rb_define_method(cVideo, "to_image", RUBY_METHOD_FUNC(to_image), 0);
-	cVideo.define_method("[]", at);
 	rb_define_module_function(cVideo, "load", RUBY_METHOD_FUNC(load), 1);
 	rb_define_module_function(cVideo, "exts", RUBY_METHOD_FUNC(exts), 0);
 }

data/.github/workflows/release-gem.yml

@@ -9,52 +9,5 @@ permissions:
 
 jobs:
   release:
-    runs-on: macos-latest
-
-    steps:
-      - name: ruby 3.2
-        uses: ruby/setup-ruby@v1
-        with:
-          ruby-version: 3.2
-
-      - name: checkout
-        uses: actions/checkout@v4
-
-      - name: setup gems
-        run: bundle install
-
-      - name: setup dependencies
-        run: "ruby -I.github/workflows -rutils -e 'setup_dependencies'"
-
-      - name: packages
-        run: bundle exec rake packages
-
-      - name: setup zig
-        if: github.event.repository.name == 'reflex-terminal'
-        uses: mlugg/setup-zig@v2
-        with:
-          version: 0.16.0
-
-      - name: test
-        run: bundle exec rake test
-
-      - name: create gem
-        id: gem
-        run: |
-          bundle exec rake gem
-          echo path=$(ruby -e 'print Dir.glob("*.gem").first') >> $GITHUB_OUTPUT
-
-      - name: create github release
-        env:
-          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        run: ruby -I.github/workflows -rutils -e 'release(*ARGV)' ./${{ steps.gem.outputs.path }}
-
-      - name: upload to rubygems
-        env:
-          GEM_HOST_API_KEY: ${{ secrets.RUBYGEMS_AUTH_TOKEN }}
-        run: |
-          mkdir -p $HOME/.gem
-          touch $HOME/.gem/credentials
-          chmod 0600 $HOME/.gem/credentials
-          printf -- "---\n:rubygems_api_key: ${GEM_HOST_API_KEY}\n" > $HOME/.gem/credentials
-          bundle exec rake upload
+    uses: xord/xot/.github/workflows/release-gem.yml@master
+    secrets: inherit

data/.github/workflows/tag.yml

@@ -1,35 +1,12 @@
 name: Tag
 
 on:
-  push:
+  workflow_run:
+    workflows: [Test]
+    types: [completed]
     branches: [master]
 
 jobs:
   tag:
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: ruby 3.2
-        uses: ruby/setup-ruby@v1
-        with:
-          ruby-version: 3.2
-
-      - name: checkout
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-          token: ${{ secrets.PAT }}
-
-      - name: setup dependencies
-        run: "ruby -I.github/workflows -rutils -e 'setup_dependencies only: :xot'"
-
-      - name: setup user name and email
-        run: |
-          git config --global user.email "xordog@gmail.com"
-          git config --global user.name "xord"
-
-      - name: tag versions
-        run: "ruby -I.github/workflows -rutils -e 'tag_versions'"
-
-      - name: push tags
-        run: git push origin --tags
+    uses: xord/xot/.github/workflows/tag.yml@master
+    secrets: inherit

data/.github/workflows/test.yml

@@ -7,40 +7,4 @@ on:
 
 jobs:
   test:
-    runs-on: macos-latest
-
-    steps:
-      - name: ruby 3.2
-        uses: ruby/setup-ruby@v1
-        with:
-          ruby-version: 3.2
-
-      - name: checkout
-        uses: actions/checkout@v4
-
-      - name: setup gems
-        run: bundle install
-
-      - name: setup dependencies
-        run: "ruby -I.github/workflows -rutils -e 'setup_dependencies'"
-
-      - name: packages
-        run: bundle exec rake verbose packages
-
-      - name: setup zig
-        if: github.event.repository.name == 'reflex-terminal'
-        uses: mlugg/setup-zig@v2
-        with:
-          version: 0.16.0
-
-      - name: vendor
-        run: bundle exec rake vendor
-
-      - name: lib
-        run: bundle exec rake verbose lib
-
-      - name: ext
-        run: bundle exec rake verbose ext
-
-      - name: test
-        run: bundle exec rake verbose test
+    uses: xord/xot/.github/workflows/test.yml@master

data/ChangeLog.md

@@ -1,6 +1,21 @@
 # rays-video ChangeLog
 
 
+## [v0.3.0] - 2026-11-04
+
+- [BREAKING] Make the frames of a video read-only views sharing one decoded frame, and take a writable copy by Image#dup
+- [BREAKING] Reject frames whose size or pixel density differs from the video's
+- Decode video files sequentially with AVAssetReader, which is many times faster
+- Add Video#get and #set, as #[] and #[]=
+- Add Video#push and #<< as aliases of #append, which returns self now
+- Show the last frame for an index past the estimated last one
+
+- Fix Video#insert crashing on an index past the end, and raise IndexError
+- Fix a crash on the error paths, as saving a GIF to a path that cannot be written
+- Fix linking the extension statically on macOS
+- Fix the build for iOS
+
+
 ## [v0.2.0] - 2026-08-04
 
 - Update dependencies

data/Gemfile.lock

@@ -0,0 +1,26 @@
+GEM
+  remote: https://rubygems.org/
+  specs:
+    power_assert (3.1.0)
+    rake (13.4.2)
+    test-unit (3.7.8)
+      power_assert
+    yard (0.9.45)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  rake
+  test-unit
+  yard
+
+CHECKSUMS
+  bundler (4.0.20) sha256=7978a8ac648767f5e635bc522445b79e80a52b907a39a36c2d8085ed6bc762ae
+  power_assert (3.1.0) sha256=fefe98ed5d3140628138f3f9f9194d6ece01f1320e69700d3c928adca7518251
+  rake (13.4.2) sha256=cb825b2bd5f1f8e91ca37bddb4b9aaf345551b4731da62949be002fa89283701
+  test-unit (3.7.8) sha256=689d1ca53f4d46f678b4e5d68d8e4294f87fc5e8b9238cc4de7c5727e8d65943
+  yard (0.9.45) sha256=52e211493f7cb8a3ebf7e104a25a1e73937a3103092545d34cb88fafebb3dc51
+
+BUNDLED WITH
+  4.0.20

data/README.md

@@ -64,20 +64,28 @@ A finite sequence of `Rays::Image` frames with a fixed `width`, `height`, `fps`,
 | `Video.new(width, height, fps:, pixel_density:)` | Create an empty video (`fps` defaults to 30)                 |
 | `Video.load(path)`                         | Load a video file from disk                                        |
 | `Video.exts`                               | Supported video file extensions on the current platform            |
-| `video.append(*images)`                    | Append one or more frames                                          |
+| `video.append(*images)`                    | Append one or more frames (also `push` and `<<`)                   |
 | `video.insert(index, *images)`             | Insert frames at the given index                                   |
 | `video.remove(index)`                      | Remove the frame at the given index                                |
 | `video.each { \|image\| ... }`             | Iterate frames (also includes `Enumerable`)                        |
 | `video[i]`                                 | Get the frame at index *i* as a `Rays::Image`                      |
+| `video[i] = image`                         | Replace the frame at index *i*                                     |
 | `video.pos` / `video.pos =`                | Current playback position (index)                                  |
 | `video.play` / `video.pause` / `video.stop`| Playback controls                                                  |
 | `video.time_scale` / `video.time_scale =`  | Speed multiplier for playback                                      |
 | `video.size`, `video.empty?`               | Frame count / emptiness                                            |
 | `video.width`, `video.height`, `video.fps`, `video.pixel_density` | Read-only metadata                          |
-| `video.dup`                                | Deep-ish copy (shares image references)                            |
+| `video.dup`                                | Copy the frame list; loaded frames get their own decoder           |
 | `video.save(path)`                         | Encode the video to a file                                         |
 | `video.to_image` (`Image()` cast)          | Get the frame at the current `pos` as a `Rays::Image`              |
 
+Every frame must have the video's `width`, `height` and `pixel_density`.
+
+Frames of a loaded video are read-only views: each source keeps a single
+decoded frame, and a frame's pixels are decoded into it on access. Such
+frames are frozen, so `paint` and `[]=` raise `FrozenError`. `dup` a frame
+to keep or edit it, and put it back with `video[i] = image`.
+
 ## 💡 Usage
 
 ### Build a video from frames and save it
@@ -129,6 +137,20 @@ video.pos = video.size / 2     # mid-point
 mid = video.to_image
 ```
 
+### Edit a frame of a loaded video
+
+```ruby
+video = Rays::Video.load 'clip.mp4'
+
+frame = video[30].dup # loaded frames are frozen, so copy first
+frame.paint do |p|
+  p.fill 1, 0, 0
+  p.ellipse 10, 10, 50, 50
+end
+video[30] = frame
+video.save 'edited.mp4'
+```
+
 ## 🛠️ Development
 
 ```bash

data/VERSION

@@ -1 +1 @@
-0.2.0
+0.3.0

data/ext/rays-video/extconf.rb

@@ -15,7 +15,8 @@ Xot::ExtConf.new Xot, Rucy, Beeps, Rays, RaysVideo do
   setup do
     headers    << 'ruby.h'
     libs.unshift 'gdi32', 'opengl32', 'glew32'            if win32?
-    frameworks << 'AppKit' << 'AVFoundation'              if osx?
+    frameworks.concat %w[
+      AppKit AVFoundation UniformTypeIdentifiers]         if osx?
     $LDFLAGS   << ' -Wl,--out-implib=librays-video.dll.a' if mingw? || cygwin?
   end