agencyenterprise/neurotechdevkit

Specify `stride` version or commit-hash

Closed this issue · 1 comments

charlesbmi commented

Issue

The current docs instruct users to install stride from the Github repository: https://github.com/trustimaging/stride . This provides NDK users with the newest stride version, with a (slight) chance that stride updates break something in NDK.

For better reproducibility, we should document/specify (e.g., via poetry.lock) the used stride version.

This is particularly relevant to replicating benchmark simulations, such as #88 and #89

charlesbmi commented

FYI, this caused an issue with a recent PR: https://github.com/agencyenterprise/neurotechdevkit/actions/runs/6433204242/job/17469780145#step:4:205
(root-caused to the issue outlined in
trustimaging/stride#57)

An alternative would be to point users to install our fork of stride: https://github.com/agencyenterprise/stride