Matteo Frattaroli

Results 10 comments of Matteo Frattaroli

Same problem here. I agree with @Basekill 's statement: > it is a general issue with agents being initialised for you in methods (internal helper agents that are not created...

@microsoft-github-policy-service agree

@ekzhu @sonichi @marklysze I completed all the changes you requested but after merging main several builds are failing because of a missing package (namely "packaging"). @ekzhu I see you submitted...

@marklysze thank you for the positive feedback! As I'm sure you noticed though there is some problem with the CI which is why I rolled back the latest merge from...

> > @marklysze thank you for the positive feedback! > > As I'm sure you noticed though there is some problem with the CI which is why I rolled back...

> There is one approach to solve the problem in a more fundamental way: Move the `register_model_client` out of the agents. Instead, make it a function to manipulate configs. Then,...

Hello again @sonichi - I was reflecting on your idea during my lunch break and looking at the code I'm not 100% sure that it would completely resolve this problem...

> > When can we expect this PR to be merged? Group chat with "auto" speaker selection does not work without this. :( > > @sourabhXIII and @Matteo-Frattaroli, I'm testing...

Hi @marklysze, I'm sorry that I couldn't get back to you before. I was just checking the code again and about the points you raised I'd say that passing in...

@bankyresearch @AryamanGurjar apparently they moved around the files or modified some __init__ files. This should work: `lp.models.detectron2.Detectron2LayoutModel` Unluckily even though I resolved the AttributeError, i ran into another issue: >...