Skip to content

Add version 2.20.1 to image package - #827

Merged
AvinoamK merged 2 commits into
gnu-octave:mainfrom
AvinoamK:main
Sep 3, 2026
Merged

Add version 2.20.1 to image package#827
AvinoamK merged 2 commits into
gnu-octave:mainfrom
AvinoamK:main

Conversation

@AvinoamK

@AvinoamK AvinoamK commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

** image 2.20.1 is a bug fix release.

** Improve image spatial filters for higher-dimensional inputs (bug #45088).

** edge: Allow binary input image for edge (bug #55432)

** mormxcorr2: re-write the code and optimize it to make it compatible
with MATLAB (bug #50151)

@mmuetzel

mmuetzel commented Sep 3, 2026

Copy link
Copy Markdown
Member

We got this twice in the CI:

Run: pkg install   image@2.20.1
  
      ERROR: package installation failed
  

After that, the CI attempts to install the package with the -verbose flag. I assume that is to get more verbose information about why the package failed installing.
But the attempt with the -verbose flag succeeds for some reason.

Any idea why it fails in the first place?

It is running this where it catches the error:

## Try to install and test the default (first) version of the changed
## packages.
try
step_group_start (["Run: pkg install ", pkg_name_version]);
pkg_install_sha256_check (pkg_info, test_dir);
step_group_end ("done.");
catch e
step_group_end ("ERROR: package installation failed");
## In case of install error, try to get as much information as possible.
##
## Note that the installation is likely to fail for packages with
## "exotic" dependencies.
step_group_start (["ERROR: pkg install -verbose ", pkg_name_version]);
pkg ("install", "-verbose", pkg_info.url);
step_group_end ("done.");
exit (1); # Return test failed.
end

@AvinoamK

AvinoamK commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

Looks like it failed with sha256 calculation, but I have checked it again, and it looks OK.

@AvinoamK

AvinoamK commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

What should I do now?

@AvinoamK
AvinoamK merged commit 998ce46 into gnu-octave:main Sep 3, 2026
2 checks passed
@AvinoamK

AvinoamK commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

Now it succeeded, so I pushed it

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