OpenMined / Threepio

A multi-language library for translating commands between PyTorch, TensorFlow, and TensorFlow.js
Other
56 stars 15 forks source link

TFJS doc parser: some commands are missing scope #113

Open vvmnnnkv opened 4 years ago

vvmnnnkv commented 4 years ago

Description

Noticed issue in tfjs parsed .json file: e.g. if you see tf.GraphModel in tfjs API, it's methods are listed w/o tf.GraphModel. prefix. Doc parser doesn't add tf.GraphModel prefix, so all these methods appear as is they were in global scope.

How to Reproduce

See tf.Tensor, tf.GraphModel, etc. in API doc https://js.tensorflow.org/api/latest/#class:GraphModel Check docs-crawler/docs/output/tfjs/1.5.1.json, notice all methods are in global scope.

Expected Behavior

Parsed data should contain indication of class where method belongs. (Or just skip all these methods because they are not under tf. but rather in specific classes, which requires target support anyway?)

Screenshots

n/a

System Information

Additional Context

Might be related to #16