Skip to content

Refactor HELIX code to simplify the implemetation#850

Merged
WenjieDu merged 5 commits into
devfrom
(refact)helix
May 3, 2026
Merged

Refactor HELIX code to simplify the implemetation#850
WenjieDu merged 5 commits into
devfrom
(refact)helix

Conversation

@WenjieDu

@WenjieDu WenjieDu commented May 3, 2026

Copy link
Copy Markdown
Owner

What does this PR do?

  1. fixing Add HELIX (ICML 2026 spotlight) #849;

Before submitting

  • This PR is made to fix a typo or improve the docs (you can dismiss the other checks if this is the case);
  • Was this discussed/approved via a GitHub issue? Please add a link to it if that's the case;
  • I have commented my code, particularly in hard-to-understand areas;
  • I have written the necessary tests and already run them locally;

@sonarqubecloud

sonarqubecloud Bot commented May 3, 2026

Copy link
Copy Markdown

@WenjieDu WenjieDu merged commit 6223cae into dev May 3, 2026
5 checks passed
@coveralls

coveralls commented May 3, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 25281713092

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage increased (+1.2%) to 80.104%

Details

  • Coverage increased (+1.2%) from the base build.
  • Patch coverage: 13 uncovered changes across 2 files (100 of 113 lines covered, 88.5%).
  • 1 coverage regression across 1 file.

Uncovered Changes

File Changed Covered %
pypots/nn/modules/helix/backbone.py 102 90 88.24%
pypots/imputation/helix/model.py 7 6 85.71%

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Losing Coverage Coverage
pypots/imputation/helix/model.py 1 90.32%

Coverage Stats

Coverage Status
Relevant Lines: 19215
Covered Lines: 15392
Line Coverage: 80.1%
Coverage Strength: 1.6 hits per line

💛 - Coveralls

@WenjieDu WenjieDu deleted the (refact)helix branch May 3, 2026 14:39
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