search

flux-framework / dyad

DYAD: DYnamic and Asynchronous Data Streamliner
GNU Lesser General Public License v3.0
6 stars 5 forks source link

Common defs and misc changes #76

Closed JaeseungYeom closed 5 months ago

JaeseungYeom commented 5 months ago
  1. Apply DYAD_DTL_DEFAULT to core init: both env parsing and dtl init. It was by default UCX in module.
  2. move common headers to common. dyad_envs.h dyad_dtl.h
  3. change the name of dyad_dtl_impl to dyad_dlt_api
  4. Removed broken urpc code
  5. dyad module to take env variable for dtl mode if command line option is not set
JaeseungYeom commented 5 months ago

@hariharan-devarajan Can you add the allocation fix?

hariharan-devarajan commented 5 months ago

@hariharan-devarajan Can you add the allocation fix?

i added that

JaeseungYeom commented 5 months ago

@JaeseungYeom Lets add the env code on the module and fix our CI?

That is in there. Looks like checks passed for both C and CPP. Not python though.

hariharan-devarajan commented 5 months ago

It seems like the site-packages installation is screwing up for root installation case using binary. I am thinking trying with a virtual env and seeing if that helps.