Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix hybrid descriptor training error #1052

Merged
merged 1 commit into from
Aug 28, 2021
Merged

Conversation

njzjz
Copy link
Member

@njzjz njzjz commented Aug 28, 2021

fix #1051.

@codecov-commenter
Copy link

codecov-commenter commented Aug 28, 2021

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.10%. Comparing base (37af1d1) to head (7691783).
Report is 1493 commits behind head on devel.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #1052      +/-   ##
==========================================
- Coverage   82.90%   75.10%   -7.81%     
==========================================
  Files         120       87      -33     
  Lines       10139     6953    -3186     
==========================================
- Hits         8406     5222    -3184     
+ Misses       1733     1731       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@amcadmus amcadmus enabled auto-merge (squash) August 28, 2021 09:00
@amcadmus amcadmus merged commit 5f8989b into deepmodeling:devel Aug 28, 2021
@njzjz njzjz linked an issue Aug 28, 2021 that may be closed by this pull request
@njzjz njzjz deleted the fix-1051 branch August 28, 2021 17:42
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.

[BUG] hybrid descriptor training failed with KeyError
4 participants