Mathieu Guay-Paquet

Results 34 comments of Mathieu Guay-Paquet

A slight correction: the formula I had in mind has the square root in a different place: $$\frac{1}{n} \sum_{slice=1}^n \sqrt{T_x[slice]^2 + T_y[slice]^2}$$ In words: for each time point and each...

This comment is just to add a cross-reference to #4204, to keep in mind this feature request while doing the refactor.

To answer the immediate question: the Frostell et al. levels were first introduced in SCT v6.1. As you noted, this happened in PAM50 release r20231024 ([changelog](https://github.com/spinalcordtoolbox/PAM50/releases/tag/r20231024)). On the SCT side,...

@joshuacwnewton says: > One way to better indicate the separation between before/after code could be to use a `try:`/`finally:` block (as shown by the [`contextmanager`](https://docs.python.org/3/library/contextlib.html#contextlib.contextmanager) docs)? Perhaps this is the...

I love the new broken link checker, but it's proving to be a little flaky (and maybe doing hundreds of web requests in the space of two minutes is a...

> * Note that trying to get the order of arguments from argparse itself [seems challenging](https://stackoverflow.com/questions/9027028/argparse-argument-order), so I think we're lucky that we have access to the unparsed arguments. For...

My first guess (but it's just a guess, [we should profile it](https://docs.python.org/3/library/profile.html)) would be that one or more of our `import` statements is heavy; probably one or more of the...

> * Could we eliminate the confusion entirely by having two columns? One for IS slice # (`'IS slice (0 = I)'`), and one for SI slice # (`'SI slice...

I think this is a great idea. A few thoughts: * Even if `bash` is available, I can imagine that some users would be confused as to which shell they're...