GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — NaiveConvolutions.convolveNaive should be forbidden from adaptation · Issue #257 · scijava/scijava

NaiveConvolutions.convolveNaive should be forbidden from adaptation #257

New issue
Copy link
New issue
Copy link
Open
Open
NaiveConvolutions.convolveNaive should be forbidden from adaptation#257
Copy link
Assignees
gselzer
Labels
bug
Milestone
1.0.1

Description

@gselzer
gselzer
opened on Aug 5, 2024
Issue body actions

This op throws an exception when currently adapted because it expects that the output, extended by the radius of the kernel on both sides in every dimension, is completely inside the input. Example:

  • Input is bounded by minimum [0, 0] and maximum [3, 3]
  • Kernel is bounded by minimum [0, 0] and maximum [3, 3]
  • Output is bounded by minimum [0, 0] and maximum [3, 3]
    This will fail because the algorithm expects the input to be queryable within minimum [-3, -3] and maximum [6, 6], throwing an error.

The solution is to prevent adaptation, and likely also to beef up the javadoc. However this will be blocked by #256

Activity

  1. gselzer
    added
    bug
    on Aug 5, 2024
  2. gselzer
    added this to the 1.0.1 milestone on Aug 5, 2024
  3. gselzer
    self-assigned this
    on Aug 5, 2024
  4. gselzer
    added this to SciJava Ops Paper #2: Polygloton Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

  • @gselzer
    gselzer

Labels

bug

Type

No type

Projects

  • SciJava Ops Paper #2: Polyglot
    Status
    No status
    Show more project fields

Milestone

  • 1.0.1No due date

Relationships

None yet

Development

No branches or pull requests

Issue actions

You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.