Adam Li

The general brain regions are known and stated in the metadata I think. The interpretation of the time series is more clinical such that certain electrodes correspond to an suspected...

> > > > iEEG data often do not make any sense if electrode positions are not shared (it is like sharing voxel timeseries from different brain regions that are...

To provide additional context. Back then, it was more difficult to pipeline the electrode locations (at least for me). That and getting a hold of the MRI/CT data in usable...

> IMHO the examples described above rather call for a RECOMMENDED status, where the validator would send a warning if no `electrodes.tsv` file is found. The warning may serve the...

> I would definitely wait for dropping 3.8 support and restrict the networkx version. Even 3.8 is not that commonly used yet. So, taking out that support would probably break...

I think numpy, [pandas](https://github.com/pandas-dev/pandas/blob/main/pyproject.toml), scipy have all removed support for Python3.8 at this point on `main`. I think scikit-learn is in the process as well too. Their general strategy is...

Yes. I think the initial work can just use pywhy-graphs. We can revisit how to refactor if necessary once we get a working prototype.

> @adam2392 Is this an ok number of unit tests? Yes to start. I didn't take a look at the details to see whether each graph is correct or not....

No problem. I've been backed up I think a good approach is to define an API similar to whatever is in dowhy currently for any type of identification. Then let's...

Just chiming in here. Yes that is what I suspect would be the best. This would require you to have a minimal example dataset that you can use for testing...