Closed anton083 closed 6 months ago
Attention: 53 lines
in your changes are missing coverage. Please review.
Comparison is base (
23e783f
) 100.00% compared to head (47edee6
) 84.31%.
Files | Patch % | Lines |
---|---|---|
src/backbone/dihedrals.jl | 59.54% | 53 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Based on #8 (i don't really know how pull requests, forks, and remotes work)
Changes to #8:
batched_mul
function through NNlib instead of Flux to avoid unnecessary indirect dependenciesbackbone/distance.jl
tobackbone/bonds.jl
backbone_bond_vectors
function inbackbone/bonds.jl
bond_lengths
andbond_angles
variables toMEAN_BOND_LENGTH
andMEAN_BOND_ANGLE
turn into consts.module Dihedrals
with hopes of making the functions work well with theBackbone
typebond_lengths
andbond_angles
into keyword argumentsget_triangle
tolaw_of_cosines
::Backbone
method toget_dihedrals
function