convexengineering / gplibrary

Useful subsystem models
MIT License
10 stars 11 forks source link

failing tests #140

Closed mjburton11 closed 6 years ago

mjburton11 commented 6 years ago

@bqpd this is the error I'm getting right from Jenkins:

Marking the build as failed.
Build was aborted
Cloning into 'solar'...
[xUnit] [INFO] - Starting to record.
[xUnit] [INFO] - Processing JUnit
[xUnit] [INFO] - [JUnit] - 4 test report file(s) were found with the pattern '**/test_reports/*.xml, **/test_reports_nounits/*.xml' relative to '/Users/jenkins/workspace/gplibrary_Push_Models/buildnode/macys/optimizer/mosek' for the testing framework 'JUnit'.

Any idea as to what is going on?

mjburton11 commented 6 years ago

Wait at another look I think it might have to do with SP_aircraft, @1ozturkbe do you know what might be going on here?

Warning: Constraint [-C_{p_{fuel}_Mission/Aircraft/Engine/Combustor*T_{t_f}_Mission/Aircraft/Engine/Combustor*f_Mission/A... <= 
\eta_{B}_Mission/Aircraft/Engine/Combustor*f_Mission/Aircraft/Engine/EnginePerformance/CombustorPerf...] 
is not tight because the left hand side eBuild timed out (after 30 minutes). 
Marking the build as failed.
Build was aborted
mjburton11 commented 6 years ago

Sorry, just talked to @bqpd and it is a time out error... so...

bqpd commented 6 years ago

Yup, these happen sometimes when other jobs are taking Jenkins slots...

mjburton11 commented 6 years ago

actually the issue is that solar is failing when it shouldn't be...

mjburton11 commented 6 years ago

@bqpd should I fix this by making solar installable?

bqpd commented 6 years ago

That might be it!

On Nov 28, 2017 19:37, "Michael Burton" notifications@github.com wrote:

@bqpd https://github.com/bqpd should I fix this by making solar installable?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/convexengineering/gplibrary/issues/140#issuecomment-347712757, or mute the thread https://github.com/notifications/unsubscribe-auth/ABagGK5aBLvKeD-iO9iZQgoinr1V2LpRks5s7KdTgaJpZM4QtYV0 .

bqpd commented 6 years ago

But it should test from within the root directory, so it may not need that

On Nov 28, 2017 20:09, Edward Burnell eburn@mit.edu wrote:

That might be it!

On Nov 28, 2017 19:37, "Michael Burton" notifications@github.com wrote:

@bqpd https://github.com/bqpd should I fix this by making solar installable?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/convexengineering/gplibrary/issues/140#issuecomment-347712757, or mute the thread https://github.com/notifications/unsubscribe-auth/ABagGK5aBLvKeD-iO9iZQgoinr1V2LpRks5s7KdTgaJpZM4QtYV0 .

mjburton11 commented 6 years ago

That didn't seem to work... see convexengineering/solar#14

mjburton11 commented 6 years ago

Why could I get it to work on solar but it doesn't work on gplibrary?

mjburton11 commented 6 years ago

thank you @bqpd!

mjburton11 commented 6 years ago

I think this refailed because it timed out before solving. Should we extend the allowable testing time?

bqpd commented 6 years ago

aiee. was it the other jobs again?

mjburton11 commented 6 years ago

I believe it was the other jobs again. On Tue, Dec 5, 2017 at 4:17 PM bqpd notifications@github.com wrote:

aiee. was it the other jobs again?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/convexengineering/gplibrary/issues/140#issuecomment-349444326, or mute the thread https://github.com/notifications/unsubscribe-auth/ANnWkV7YnM8KZdLqKZe6BZgkpEWpMj4vks5s9bLtgaJpZM4QtYV0 .

-- Michael Burton MIT Aero/Astro Ph.D. Student mjburton@mit.edu

mjburton11 commented 6 years ago

Got the same issue again Build timed out (after 30 minutes). Marking the build as failed. Build was aborted

bqpd commented 6 years ago

added 10 minutes, timeout is now 40...let's see if that improves its success rate sufficiently

bqpd commented 6 years ago

actually increased to 60, because that's what the models test PR was already at :p