classifier-reborn icon indicating copy to clipboard operation
classifier-reborn copied to clipboard

Deprecate magic train untrain methods

Open ibnesayeed opened this issue 8 years ago • 10 comments

Before releasing 2.2.0 magical train_* and untrain_* style methods should be deprecated.

  • [x] Add deprecation warning
  • [ ] Add tests to not use these methods
  • [x] Update documentation to not use these methods

ibnesayeed avatar Jan 18 '17 20:01 ibnesayeed

It would also be good to have the official development Docker image available (#101) before the release of 2.2.0 so that it can be documented and advertised with this rather significant release.

ibnesayeed avatar Jan 18 '17 20:01 ibnesayeed

Perhaps we can also include basic validation #71 as part of 2.2.0 release.

ibnesayeed avatar Jan 18 '17 20:01 ibnesayeed

Sounds good. I have a working branch for the first two items. I'll push it up later.

Ch4s3 avatar Jan 18 '17 21:01 Ch4s3

I can take care of the documentation, later tonight.

ibnesayeed avatar Jan 18 '17 21:01 ibnesayeed

PR #141 takes care of the documentation.

ibnesayeed avatar Jan 19 '17 00:01 ibnesayeed

@Ch4s3 a note for you. I have seen test_external_classifying of lsi_test.rb using magic methods. Just make sure to take care of all the places where Bayes is being called with magic methods.

ibnesayeed avatar Jan 21 '17 22:01 ibnesayeed

Yeah, that's on the list. I'll try to devote some time to this tomorrow or Monday.

Ch4s3 avatar Jan 21 '17 22:01 Ch4s3

I'll try to get back to this asap

Ch4s3 avatar Feb 08 '17 20:02 Ch4s3

@Ch4s3 please take care of the second point on the original post of this ticket before releasing 2.2.0 as per #164. This should be pretty easy and quick to deal with.

ibnesayeed avatar Aug 01 '17 03:08 ibnesayeed

@ibnesayeed will do.

Ch4s3 avatar Aug 02 '17 04:08 Ch4s3