deepsegment icon indicating copy to clipboard operation
deepsegment copied to clipboard

Add the ability to change the dimension

Open nagadit opened this issue 5 years ago • 3 comments

I have glove 300d vectors only. Change pls code here and here to this: def train(x, y, vx, vy, epochs, batch_size, save_folder, glove_path, dim_glove=100) and model = seqtag_keras.Sequence(embeddings=embeddings, word_embedding_dim=dim_glove, word_lstm_size = dim_glove) tnx

nagadit avatar Feb 26 '20 15:02 nagadit

@good-mouse 300d vectors are mostly a overkill and will result in at least 4x performance degradation.

If you really need support for 300d vectors inside the library, please consider making a PR. I will be more than happy to help if you need any guidance for that.

bedapudi6788 avatar Feb 26 '20 18:02 bedapudi6788

  1. Fork the repo.
  2. create a new branch
  3. Make your changes and push to your forked repo.
  4. Create a pull request.

https://yangsu.github.io/pull-request-tutorial/

bedapudi6788 avatar Feb 26 '20 19:02 bedapudi6788

@bedapudi6788 I thought you mean PR as an advertisement 😄

nagadit avatar Feb 26 '20 19:02 nagadit