Fix/various issues

[mclarsen]

No description provided.

[cyrush]

@mclarsen hola is hanging b/c mesh bp verify is failing on at least one of the ranks, causing that MPI task to crash and the others to wait for eternity

[cyrush]

i understand the hola issue, my spack changes made conduit 0.3.1 build again, instead of conduit@master

[mclarsen]

@cyrush I had to resolve the undefined symbol to register filter that occurring when vtkh was off and python wanted to see register_filter. Instead of a quick fix, I am going to compile the ascent runtime without vtkh. To support this the right way, I will need to change some of the tests to look at a new about variable that indicates if vtkh is enabled.

[cyrush]

looks like its still picking up wrong conduit for the main case, perhaps it has to do with +mfem

[cyrush]

mfem requires conduit version 0.3.1:, but spec asked for master,

I thought the mfem constraint was 0.3.1 or greater, but obviously something is wrong -- I'll try to fix this soon.

[mclarsen]

I'm out during the day today but I'll finish my runtime changes tonight. My goal is to get this merged this weekend as it clears a lot of issues

…

On Sat, Sep 8, 2018, 2:22 PM Cyrus Harrison ***@***.***> wrote: mfem requires conduit version 0.3.1:, but spec asked for master, I thought the mfem constraint was 0.3.1 or greater, but obviously something is wrong -- I'll try to fix this soon. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#146 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AD2d8YvmkrqO77dE7hwFN5OrvM5RBHeAks5uZDUcgaJpZM4WfrZi> .

[cyrush]

I believe the fix is easy, I’ll try to push something tonight On September 8, 2018 at 2:30:25 PM PDT, Matt Larsen <notifications@github.com> wrote: I'm out during the day today but I'll finish my runtime changes tonight. My goal is to get this merged this weekend as it clears a lot of issues

On Sat, Sep 8, 2018, 2:22 PM Cyrus Harrison ***@***.***> wrote: mfem requires conduit version 0.3.1:, but spec asked for master, I thought the mfem constraint was 0.3.1 or greater, but obviously something is wrong -- I'll try to fix this soon. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#146 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AD2d8YvmkrqO77dE7hwFN5OrvM5RBHeAks5uZDUcgaJpZM4WfrZi> .

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#146 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ABI6HkyIR9zr9K0QzYB5hoQlE5AbIOQtks5uZDbmgaJpZM4WfrZi>.

[mclarsen]

@cyrush Do the ascent runtime changes look ok to you?

[mclarsen]

This allows us to test extracts without having to compile in vtkm or vtkh

[cyrush]

glanced at the changes -- looks good to me!

[mclarsen]

OK. We be green again.

[cyrush]

Nice, thanks for tackling all of those issues!