Garrett Michael Flynn
Garrett Michael Flynn
You mean this PR? https://github.com/NeurodataWithoutBorders/nwb-guide/pull/530
Either one is fine. Depends on how comprehensive we want to be (e.g. we could quite easily run through all the YAML pipelines automatically). It would be great if all...
#530 still requires that the testing data is on the user's computer, so we'll still have to install on the CI in a default location. You're right, though, we will...
> That's pretty tough to do in a way that scales well (even in actions we utilize caching) and is really interoperable Got it, thank you for the clarification. I'll...
Oh I see, I read your changes incorrectly. `base_path` is for where the generated data will be _output_. Do you want to run E2E tests using the testing data at...
Got it. Changed the name of this PR and will keep it in draft mode. I'll work on #530 until merged, then continue to build out #582.
Hmm it looks _possible_ but could be somewhat complicated. Having some issues figuring it out because of [blocked tests after adding the DANDI_CACHE environment variable](https://github.com/NeurodataWithoutBorders/nwb-guide/pull/623#issuecomment-1957658027)—but I'll try a few things...
Technically no—though we have pretty much decided not to pursue this. This particular PR prepares the E2E tests to run through _all_ of the generated test pipelines. Something outside the...
Yeah I was thinking this too—even just to have the full suite available for a local test. Seems like both approaches are useful and on the roadmap. In that case,...
The general workflow is complete, though there are now a few additional issues that have been uncovered. In summary: 1. Suite2P + Caiman: Fluorescence data is not handled correctly -...