Sofie Van Landeghem

Results 361 comments of Sofie Van Landeghem

@pmbaumgartner : yes - that was indirectly the point of my question: it's unclear to a reader how to define these dictionaries and they'll end up trying to use the...

> I think that `set_store_activations` is a really confusing name. Are you setting? Are you storing? Are you setting "store activations"? What are "store activations"? Are they "stored activations"? 😁...

I like this idea - could it be ``` spacy info en_core_web_sm --url ``` that would basically work regardless of whether the model is already installed or not?

I'd prefer having the docs changes with the PR!

Good point/question, but I agree with Adriane that this is generic enough to be in the main library. Like I wouldn't want to add 3 more loggers here, but one...

> Is there a good way to write unit tests for these kinds of things? I've recreated all possible cases locally, but I think it might be better if we'd...

Hi @shen-qin, we really appreciate the time and effort you've put into this PR, but I'm afraid we won't be merging it as such. After some internal discussion, we've come...

Status March 1: - Wrote preliminary v3-compatible framework to facilitate experimentation with different coref models - Currently assuming two different pipeline components: - `coref_er` / `CorefEntityRecognizer` is a rule-based mention...

Apologies for the silence on this! We have deprioritized this work for a few months, but are now picking it back up. We are actively exploring two coref solutions: the...

Hi! This is work in progress. We've got open issues and comments on this PR and to ensure those remain visible and we can focus on the technical implementation, I've...