timryanb commented 2 years ago

Fixing bug in TLOAD2 get_load_at_time method which was setting both frequency and force vector to the same variable

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts and fixes 6 when merging cd4cd63edac4561e3eb7e65768eecc2131c7c778 into 0c2b8e6103b4629087f9d5f96e799289519bfb2c - view on LGTM.com

new alerts:

2 for Module is imported with 'import' and 'import from'

fixed alerts:

5 for Unused import
1 for Variable defined multiple times

codecov[bot] commented 2 years ago

Codecov Report

Merging #703 (cd4cd63) into main (0c2b8e6) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #703   +/-   ##
=======================================
  Coverage   75.25%   75.25%           
=======================================
  Files         519      519           
  Lines      178920   178920           
  Branches    24052    24052           
=======================================
  Hits       134640   134640           
  Misses      38538    38538           
  Partials     5742     5742

Impacted Files	Coverage Δ
pyNastran/bdf/cards/loads/dloads.py	`73.53% <100.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0c2b8e6...cd4cd63. Read the comment docs.

SteveDoyle2 / pyNastran

Fixing bug in TLOAD2 get_load_at_time method #703

Codecov Report