Skip to content
This repository was archived by the owner on Jan 15, 2024. It is now read-only.

Conversation

sxjscience
Copy link
Member

Here's the script I used to plot the figures that explain different bucketing strategies (The figures in http://gluon-nlp.mxnet.io/api/notes/data_api.html). I choose to upload it here in case the others would like to use it in the future.

@sxjscience sxjscience requested a review from szha as a code owner August 29, 2019 00:38
@codecov
Copy link

codecov bot commented Aug 29, 2019

Codecov Report

❗ No coverage uploaded for pull request head (plot_bucket@f7d0285). Click here to learn what that means.
The diff coverage is n/a.

@codecov
Copy link

codecov bot commented Aug 29, 2019

Codecov Report

Merging #908 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #908   +/-   ##
=======================================
  Coverage   90.48%   90.48%           
=======================================
  Files          66       66           
  Lines        6400     6400           
=======================================
  Hits         5791     5791           
  Misses        609      609

@sxjscience
Copy link
Member Author

For the color combinations, I manually selected from https://coolors.co/ .

@mli
Copy link
Member

mli commented Aug 29, 2019

Job PR-908/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-908/1/index.html

Copy link
Contributor

@leezu leezu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! There's a typo in the filename though. Can you rename it?

@mli
Copy link
Member

mli commented Sep 2, 2019

Job PR-908/4 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-908/4/index.html

@eric-haibin-lin
Copy link
Member

Nice. If we can take a BucketSampler as the input and output the visualization, it will be even easier for other to use.

@sxjscience sxjscience merged commit e05288b into dmlc:master Sep 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants