gluon-nlp icon indicating copy to clipboard operation
gluon-nlp copied to clipboard

Difference in naming convention between op.py and test_op.py

Open AetherPrior opened this issue 4 years ago • 0 comments

Description

Mxnet's NumPy is being imported as np in the file src/gluonnlp/op.py:
https://github.com/dmlc/gluon-nlp/blob/5d4bc9eba7226ea9f9aabbbd39e3b1e886547e48/src/gluonnlp/op.py#L8

While NumPy is imported as np in test_op.py:
https://github.com/dmlc/gluon-nlp/blob/5d4bc9eba7226ea9f9aabbbd39e3b1e886547e48/tests/test_op.py#L1

This requires a change in conventions between tests and code, which can be a minor nuisance.

What have you tried to solve it?

  1. While a simple refactoring is possible, I don't want to be reverting any design decision in any way.

AetherPrior avatar Aug 24 '21 20:08 AetherPrior