Steele Farnsworth

Refactorings

2

I made a couple of non-spec changes to the BiLSTM and BERT code while doing a code exploration. Only one is intended to boost performance. I also migrated the tests...

Ensures that the CPU is used by default.

Convert the config JSON to a YAML file

3

YAML files appear to be better suited for storing configuration data than JSON. -[Config file](https://github.com/NLPatVCU/medaCy/blob/master/config.json) -- Should be converted to a YAML file. -[Config reader](https://github.com/NLPatVCU/medaCy/blob/master/medacy/tools/read_config.py) -- This should probably be...

Currently, it is incumbent on the user to know which pipeline configuration they used when creating a model. If one loads a model that they had saved, and do not...

https://github.com/allenai/scibert/blob/ae9f98f58a9793781148ccf715b03f7160060f24/scibert/models/bert_text_classifier.py#L37 BertTextClassifier requires an instance of TextFieldEmbedder and calls `get_output_dim` on that instance, however that method always raises NotImplementedError. Is this meant to deliberately un-implement a method from the superclass,...

https://github.com/allenai/scibert/blob/ae9f98f58a9793781148ccf715b03f7160060f24/scibert/models/bert_text_classifier.py#L27 I believe that this line should read `verbose_metrics: bool = False`. As it stands, it doesn't have a default value.

Revert to previous txt file deletion behavior

7

Prior to the filtering system rewrite, if a user entered a message that was too long, the bot would delete the message and respond with this message in the channel...

Implement a better system for keeping track of alternate accounts

5

Currently, if we believe that two or more accounts belong to the same person, we issue a note infraction for each account stating which other account IDs we believe to...

Proposed changes regarding generics and symmetry with dictionaries

5

I have a few changes in mind, but I wanted to verify that these are changes we'd want before I potentially clutter the commit history with them. ## Generics These...

Warning infractions in mod channels should fail if the user has DMs off

3

Currently, if a moderator issues a warning to a user in a channel that that user can't see (usually a moderator-only channel), and that user happens to not receive DMs...