Repository navigation
Conversation
Rewrite the statements the 0.5.x refactors made false: conversion is an ordered family of routes whose one predicate plans and performs, not 36 generated methods; out-of-memory handling retries on the same device and re-raises, with no CPU fallback; the decorators do not convert arguments; dtype restoration rescales to the full integer range. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
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.
Brings
paper.mdin line with the code at v0.5.1 (6f69ea4, which is alsomain). No library code changes. Paths below are undersrc/arraybridge/.Nothing was executed: the machine was loaded and no environment was created. Every name and call signature in the code examples was checked by reading the source (
ArrayLibrary.namedlibraries.py:292,convert_to:624,ConversionRoute.plannedconversion.py:124,ConversionCounters:470 withinstalled:426 andformat_table:545,AcceleratorDevicedevices.py:28, decorator lookup__init__.py:54and decorators.py:486). The examples were not run.Claim ledger
@cupy)convert_to/convert_memory; the paper now says soMemoryTypeand conversion between every pair are derived from the classes. No count is givenCupyLibrarylistingto_numpy(),from_numpy(),from_dlpack(), andconvert_to()"from_dlpack(onlyDLPackImportlibraries: NumPy and pyclesperanto have none)ArrayLibrarysubclass"threading.local()model(image)runs on thread-local stream" (the@torchexample)gc.collect()runsrun_with_oom_recoverylistingscale_dtypedoes the scalingtorchdecoratorashlar_main_gpu.py:859,basic_processor_cupy.py:285,cupy_processor.py:792,n2v2_processor_torch.py:311)New statements, with their source: the five routes and their order (conversion.py:180, 214, 244, 306, 341; CHANGELOG 0.5.1 "Conversion"); one predicate plans and performs (
candidates:108-121 is used by bothplanned:137 andrun:153); records name the mechanism used and the declined ones (:160-175, :358-372);ConversionCounters(:470); devices as values (devices.py); a host call does not initialise CUDA (libraries.py:936-941, CHANGELOG 0.5.1).False or misleading: 9 (rows 3, 6, 8, 9, 11, 13, 16, 21, 22) plus the same wording in the AI disclosure (26). Removed as unverifiable: 3 (rows 7 clause, 17, 25).
For the owner
gc.collect()(onlyCudaLibraryoverridescleanup_loaded).arraybridge>=0.4.1,<0.5(pyproject.toml:110on the branch I read), so the Research Impact section describes OpenHCS on 0.4.x while the paper describes 0.5.1.paper.bib: themetaclassregistryURL pointed atgithub.com/trissim/metaclass-registry; changed togithub.com/OpenHCSDev/metaclass-registry(confirmed to exist). The six library entries already in the bib are now cited at first mention. No reference was added.wc -w(front matter and code included) against 1019 before. The two new mechanisms (routes, records) account for the difference.🤖 Generated with Claude Code