search

CASL / Futility

VERA Fortran Utilities
Other
44 stars 20 forks source link

Update requirements table #184

Closed bkochuna closed 4 years ago

bkochuna commented 5 years ago

I'm tackling these now Bob.

When I'm done with that, I"ll email you with some more details on how to get the cdash test names file.

rks171 commented 5 years ago

There's no option to switch between latex output and html. Was the html output something we wanted to retain?

rks171 commented 4 years ago

Any update on this? I'm going through updating my requirements table and found this is still on a branch.

bkochuna commented 4 years ago

Trying to find the time to add a test. That's the only hold up on the merge.

bkochuna commented 4 years ago

@rks171 Can you go back through and resolve some of the issues?

@lamilamilami has added some testing (I'm reviewing now).

rks171 commented 4 years ago

This all looks good to me. I ran it again right now and it worked as expected for CTF.

bkochuna commented 4 years ago

Alright, Yuxuan, can you figure out what went wrong with this pipeline? We'll wait to merge this until we get Matt's issue on the Native Solvers resolved..

bkochuna commented 4 years ago

Sorry should have put @lamilamilami instead of Yuxuan

yl5 commented 4 years ago

I tried standalone build of Futility, but the new test is passing on highpoint. Is there a way to reproduce the build? Maybe a missing python library (e.g., pandas)? From the running time, it looks like the test failed as it started. @bkochuna

bkochuna commented 4 years ago

@lamilamilami if you update this branch with latest master, the CI should pass.

yl5 commented 4 years ago

I've made a couple of additional changes in the script. 1) The dependence of IPython package is commented out since this feature was deactivated in the script. This avoid installing the additional package on travis CI which is not being used by the script. 2) I sort the paths of all files so the order of them in the requirement table is consistent now. Hopefully this won't affect intended usability of generating requirement tables for MPACT and CTF @bkochuna @rks171 . Without sorting it it's really hard to test multiple inputs (a lot of combinations for generating the gold files). The CI should pass now.