ReproNim / reproschema-py

Apache License 2.0
2 stars 8 forks source link

update conversion tools and their tests #37

Closed yibeichan closed 7 months ago

yibeichan commented 7 months ago

Updated reproschema2redcap and redcap2reproschema and corresponding tests

The redcap2reproschema one has the most updates, thanks to Evan's feedback on the bridge2ai conversion

@djarecka based on Friday's discussion, I added more checks/assertions in the tests; also moved test outputs to tmpdir After you review this PR, we can talk about what else needs to be improved.

djarecka commented 7 months ago

@yibeichan - I've checked that the tests work for me and the output is created this time.

Perhaps we can just merge this and I will try to change the code to use the classes from model.py in a way that the output of the tests stays the same.

yibeichan commented 7 months ago

okay, merged! let's see how it goes once you change model.py