Title: NaiveConvolutions.convolveNaive should be forbidden from adaptation · Issue #257 · scijava/scijava · GitHub
Open Graph Title: NaiveConvolutions.convolveNaive should be forbidden from adaptation · Issue #257 · scijava/scijava
X Title: NaiveConvolutions.convolveNaive should be forbidden from adaptation · Issue #257 · scijava/scijava
Description: 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...
Open Graph Description: 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...
X Description: 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...
Opengraph URL: https://github.com/scijava/scijava/issues/257
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"NaiveConvolutions.convolveNaive should be forbidden from adaptation","articleBody":"[This op](https://github.com/scijava/scijava/blob/52b6201352d8cb647c8d7ddda086414d371d0fce/scijava-ops-image/src/main/java/org/scijava/ops/image/filter/convolve/NaiveConvolutions.java#L122) 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:\r\n* Input is bounded by minimum [0, 0] and maximum [3, 3]\r\n* Kernel is bounded by minimum [0, 0] and maximum [3, 3]\r\n* Output is bounded by minimum [0, 0] and maximum [3, 3]\r\nThis will fail because the algorithm expects the input to be queryable within minimum [-3, -3] and maximum [6, 6], throwing an error.\r\n\r\nThe solution is to prevent adaptation, and likely also to beef up the javadoc. However this will be blocked by #256 ","author":{"url":"https://github.com/gselzer","@type":"Person","name":"gselzer"},"datePublished":"2024-08-05T23:36:18.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/257/scijava/issues/257"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:8cd98886-14bb-1b6d-4c7d-efc0313eaa5c |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 9C28:DE287:15B5A73:1CF034A:696AE390 |
| html-safe-nonce | b4f62a420c86a7ca3433edd4857397c6aa9e58e6e3615583a5be4638fc2c1670 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5QzI4OkRFMjg3OjE1QjVBNzM6MUNGMDM0QTo2OTZBRTM5MCIsInZpc2l0b3JfaWQiOiI3NzQ1ODM3ODU4MDA1MjQ2ODkiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 0e8501d9ed657dff122ec36a3f2beb4a9b0fa8822c1fe54bb67f244d4abefef5 |
| hovercard-subject-tag | issue:2449645782 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/scijava/scijava/257/issue_layout |
| twitter:image | https://opengraph.githubassets.com/7ee0f65674c0719b230cc0cac70cf69a4b0ae4e9fd6f9583ca0a0c6b991c8e27/scijava/scijava/issues/257 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/7ee0f65674c0719b230cc0cac70cf69a4b0ae4e9fd6f9583ca0a0c6b991c8e27/scijava/scijava/issues/257 |
| og:image:alt | 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... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | gselzer |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5f99f7c1d70f01da5b93e5ca90303359738944d8ab470e396496262c66e60b8d |
| turbo-cache-control | no-preview |
| go-import | github.com/scijava/scijava git https://github.com/scijava/scijava.git |
| octolytics-dimension-user_id | 1262770 |
| octolytics-dimension-user_login | scijava |
| octolytics-dimension-repository_id | 85618439 |
| octolytics-dimension-repository_nwo | scijava/scijava |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 85618439 |
| octolytics-dimension-repository_network_root_nwo | scijava/scijava |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 3d84d50b3c75fa36755c3cf392edbc09e626f979 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width