Irfan Alibay

Results 475 comments of Irfan Alibay
trafficstars

> Actually I'm going to block here on CI issues, @jaclark5 can you reproduce these locally? Ha looks like numpy 1.24 just got released and we had just been ignoring...

Went ahead and merged - should be good to merge once CI is green

> The error is: testsuite/MDAnalysisTests/datafiles.py:288:89: E501 line too long (89 > 88 characters) [flake8] > > This line isn't part of this contribution, but should I fix it anyway? Nope,...

I'm just going to cycle CI again since we just merged a bunch of stuff, if that returns green I think it's good to go.

Honestly with the new 1.80 pin we need to have here, I feel like we should act on this sooner than later. Are there any objections for getting this done...

> Btw, how do we declare that an optional dependency requires a minimal version number? Had a quick look with #4332 and it doesn't look like setuptools enforces a minimum...

@BradyAJohnston for arbitrary frame specific data we do have the auxiliary system, that probably would be the best place to fetch & store such data if looking to interfacing formally...

Dynamically updating a topology is a behaviour change I doubt we could allow any time soon (it's a pretty big overhaul of how we expect things to work), is there...

``` but as it's already common practice for a lot of people to use the (now deprecated) .pdb format for storing arbitrary data in the B-factor column, ``` Herein lies...