Irfan Alibay
Irfan Alibay
So my initial view here, echoing @lilyminium and @orbeckst is that we have to be really careful about maintenance and specifically writers becoming increasingly dissimilar to each other. 1. I...
> It seems I caused more problems by bringing up my application with variable atoms. I wouldn't necessarily say it's "problems", it's good to know that these applications exist, and...
Because adding new readers is probably going to be a slow process I'd just go for the `NotImplementedError` in the short term and review it in the long term. This...
This is an interesting one. We treat things like the occupancy and bfactors main attributes as a "topology" things, so they don't don't get updated as you read a new...
> Hmm I thought this was supposed to merge upon approval, do I need to do anything else? I was just waiting on CI, I'll merge now (we don't have...
Thanks @jfennick - since there are no related issues and it'll be hard to test the error, could you explain in more details what was the cause for this fix...
Sorry about this @jfennick - I didn't get much headway on the alluded to blog post (fingers crossed end of this week). I'm dismissing my block, but could I ask...
@zemanj you have a blocking review here, could you have another look please?
> Is there anything else I need to do to get this merged? If licensing issues are impeding drive-by contributions, then in the future perhaps I'll simply submit Issues instead...
Sorry about the last two commits, just thought it was faster for me to just add you to the changelog and fix the merge conflicts in one.