Matthew Carbone
Matthew Carbone
@mathavraj this is a blast from the past. I haven't worked on Talos in a while, and it doesn't appear to be maintained actively right now. With that said, you're...
> Thank you for the prompt reply. I hope someone with merging powers would accept it. I have raised a similar pull request in wrangle also and currently in our...
@munrojm no problem! It's strange, I think the query is constructed properly. Might be a problem with the query engine itself? Let me know if there's anything I can do!
@aakhiltayal to be clear, this only occurs when there's a single absorbing atom in the unit cell, correct?
No I mean a single _absorbing atom_ in the unit cell. Based on the structure, it looks like there's a single Fe/unit cell. I don't think this problem occurs when...
Happy to attempt a fix if you'd like.
@ziatdinovmax I see what you're saying but ultimately doesn't it make more sense from a design standpoint to have a common interface? It's going to make building tools on top...
@ziatdinovmax yup sounds good. To be clear this would actually be a backwards-incompatible change (technically), since the shape of the `predict` method would change. Should we make such a change,...
Perfect, I really like the idea of `sample_from_posterior()`.
@ziatdinovmax hey Maxim, I just realized it's been a while since I posted this and I wanted to be sure you saw this: the pipeline is currently using the legacy...