fix bug with duplicate library reactions

[mjohnson541]

This sets the rxn.family attribute to the name of the library it was loaded from rather than the library it was originally from this prevents reactions in a seed mechanism that were originally from a reaction library from being duplicated if that reaction library is used as a reaction library or seed mechanism.

[alongd]

I guess you guys already discussed it, but seems like we're misusing the family attribute. Also, won't that end in a weird comment in the mechanism file, such as From reaction family: KnownLibrary?

[mjohnson541]

The comments will be the same. It isn't exactly a misuse, LibraryReaction already had a family attribute that was identical to the original library, now the family attribute is the library it was loaded from. That doesn't exactly make it a clear solution to the problem, but it does make sense in context.

[codecov]

Codecov Report

Merging #1676 into master will decrease coverage by <.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master    #1676      +/-   ##
==========================================
- Coverage   41.81%   41.81%   -0.01%     
==========================================
  Files         177      177              
  Lines       30206    30207       +1     
  Branches     6252     6252              
==========================================
  Hits        12630    12630              
- Misses      16654    16655       +1     
  Partials      922      922

Impacted Files	Coverage Δ
rmgpy/data/kinetics/library.py	43.1% <33.33%> (-0.11%)	⬇️

---

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 e9a650d...ac6162f. Read the comment docs.

[mliu49]

For documentation purposes, the second commit is to address issues I've had with loading reaction libraries with the comment Originally from reaction library: Unclassified, which is appended whenever one imports a Chemkin file in as a reaction library.

I think it's safer to only do the additional parsing for autoGenerated libraries.

[mjohnson541]

@alongd does this solve your issue?

[alongd]

Thanks for the fixes and explanations!