ModelSEEDDatabase icon indicating copy to clipboard operation
ModelSEEDDatabase copied to clipboard

checking duplicates in Travis

Open samseaver opened this issue 8 years ago • 3 comments

Can we limit all checks in Travis to non-obsolete reactions/compounds?

samseaver avatar Nov 08 '17 20:11 samseaver

I thought I was doing that... Is it not?

JamesJeffryes avatar Nov 08 '17 21:11 JamesJeffryes

I didn't know either way, but I'd assumed, given the vast number of duplicate names, that this might've been the case. Can you confirm?

samseaver avatar Nov 08 '17 21:11 samseaver

I exclude obsolete from checking but you may have a lot of comps named "NULL" or "". I may have to check that. https://github.com/ModelSEED/ModelSEEDDatabase/blob/master/Scripts/Validation/Validate_Compounds_JSON.py#L24

On Wed, Nov 8, 2017 at 3:06 PM, samseaver [email protected] wrote:

I didn't know either way, but I'd assumed, given the vast number of duplicate names, that this might've been the case. Can you confirm?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ModelSEED/ModelSEEDDatabase/issues/84#issuecomment-342959519, or mute the thread https://github.com/notifications/unsubscribe-auth/AIGfOeJSCY1yVCCYjmnEc43MJQzFFyOjks5s0hfTgaJpZM4QXB5m .

JamesJeffryes avatar Nov 08 '17 21:11 JamesJeffryes