Skip to content

Main#137

Open
seanxia8 wants to merge 2 commits into
DeepLearnPhysics:mainfrom
seanxia8:main
Open

Main#137
seanxia8 wants to merge 2 commits into
DeepLearnPhysics:mainfrom
seanxia8:main

Conversation

@seanxia8

Copy link
Copy Markdown

Description

Avoid jittering into the same coords and duplicating per coords keys

Motivation and Context

The old jitter would cause ME deconv'ed tensor to a different size from the input at the same resolution.

Fixes #(issue)

Type of Change

Please delete options that are not relevant.

  • [ Bug fix (non-breaking change which fixes an issue) ]
  • [ Documentation update ]

How Has This Been Tested?

Describe the tests you ran to verify your changes. Provide instructions so reviewers can reproduce.

  • [pytest spine/test/test_io/ -v -k "jitter or augment"]

Test Configuration:

  • OS: ["Ubuntu 22.04.3 LTS]
  • Python version: [Python 3.10.12, pytest-8.3.5, pluggy-1.5.0]
  • SPINE version: [main branch, commit c4d3b13]

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Screenshots / Event Displays (if applicable)

Add any relevant figures, plots, or event displays that demonstrate the changes.

Additional Notes

Any additional information that reviewers should know.

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.

1 participant