search

trendscenter / coinstac

Collaborative Informatics and Neuroimaging Suite Toolkit for Anonymous Computation
MIT License
45 stars 19 forks source link

Simulator miss-sets CWD for CommandComputation #38

Closed swashcap closed 8 years ago

swashcap commented 8 years ago

coinstac-simulator doesn’t properly resolve the cwd for CommandComputations.

swashcap commented 8 years ago

This is proving more complicated than initially expected. Unfortunately, due to tight coupling between the computation registry within coinstac-common, I think we’ve relied on being explicit with cwd in the declaration. This potentially confuses for algo authors: their setup should retain path info within their directory as expected. Authors shouldn’t need to add redundant cwd properties to their computations or declarations.

I’m looking into an easy fix, but this might be a rabbit hole. ??

[--- Commented from Asana.com

commenter coins

---[aa]