libCEED Travis CI With OCCA Now Failing

[jeremylt]

A recent update to OCCA now gives a failure in Travis for OCCA for all examples, CEED, MFEM, and PETSc. I suspect there is a change that needs to get updated in our OCCA backends

PR Link with failing Travis:
CEED/libCEED#143

[jeremylt]

This commit works with libCEED

527494c

[dmed256]

My bad, accidentally pushed a bad commit to master

[jeremylt]

No worries. I just wanted to make sure you knew because it could have been something we needed to adjust on our end.

[jeremylt]

I'm still seeing the issue. I made a PR for libCEED that runs Travis against the last version of OCCA that libCEED passes with.

CEED/libCEED#145

[dmed256]

Oops, misread the hash. Thought it was the HEAD commit at the time.
I'll check this tonight, my bad

[jeremylt]

No worries, thanks for looking into it

[dmed256]

@jeremylt seems the new kernelHash method is messing something up

https://github.com/libocca/occa/blob/master/src/device.cpp#L442

Changing it back to occa::hash(allProps) seems to make libCEED tests pass again. I'll keep looking to find out the root cause, just wanted to update you on that

[v-dobrev]

By the way, @dmed256, I think this issue also affects mfem's engines-dev branch and probably occa-dev as well.

[jeremylt]

Thanks for the update!

[dmed256]

@jeremylt @v-dobrev Just tested libCEED with origin/master (1450f74) and it passes again

[v-dobrev]

Thanks, David! I'll give it a try with mfem.

[v-dobrev]

The latest commit (780a06a) works OK with mfem (engines-dev).