erwanp / qtplaskin

A graphical tool to explore ZdPlaskin Results
GNU Lesser General Public License v3.0
7 stars 5 forks source link

two letters atoms problem correction #15

Closed maillardj closed 3 years ago

maillardj commented 4 years ago
  1. Modification in database.py to prevent failure when molar mass is unknown + test
  2. Modification in modeldata.py to be able to recognize two-letters atoms + test on Ar
codecov[bot] commented 4 years ago

Codecov Report

Merging #15 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #15   +/-   ##
=======================================
  Coverage   23.93%   23.93%           
=======================================
  Files          19       19           
  Lines        1780     1780           
=======================================
  Hits          426      426           
  Misses       1354     1354           

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 55f7e19...55f7e19. Read the comment docs.

erwanp commented 4 years ago

Hello ! 2 small Git problems happened :

It will be quite hard to clean the history. What I suggest instead :

erwanp commented 3 years ago

Fixed in #16