Replace subprocess calls with datalad API commands, when possible

PennLINC / CuBIDS

Curation of BIDS (CuBIDS): A sanity-preserving software package for processing BIDS datasets.

https://cubids.readthedocs.io/

MIT License

19 stars 9 forks source link

Replace subprocess calls with datalad API commands, when possible #291

Closed tsalo closed 8 months ago

tsalo commented 8 months ago

In a few places, CuBIDS does subprocess.run(["datalad", command, extras]) instead of datalad.api.<command>(extras). It seems like we could simplify the code a little bit by switching to the Datalad Python API, when possible.

mattcieslak commented 8 months ago

we might actually want to go the other direction - the datalad python api is pretty tricky

tsalo commented 8 months ago

Ah, I wasn't aware of that. That's good to know.