Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[MXNET-548] fixed path for auto_module_index.js #11287

Merged
merged 2 commits into from
Jun 29, 2018

Conversation

aaronmarkham
Copy link
Contributor

@aaronmarkham aaronmarkham commented Jun 14, 2018

Description

Fixes #11238

Checklist

Essentials

Please feel free to remove inapplicable items for your PR.

Changes

Updated the path for a js file that was triggering 404s.

The following error is what you see on production today:
2018-06-28_18-00-18

I cherry-picked this commit and added here:
http://35.170.70.99/api/python/metric/metric.html
And if you inspect the console the error is gone now that it can find this file.

I used the same pattern on the rest of the file updates.

@aaronmarkham aaronmarkham requested a review from szha as a code owner June 14, 2018 18:53
@anirudhacharya
Copy link
Member

@aaronmarkham please re-trigger the CI. It failed on a flaky test.

@aaronmarkham
Copy link
Contributor Author

5th restart due to flaky CI...

@szha
Copy link
Member

szha commented Jun 29, 2018

@aaronmarkham could you do a rebase? some tests are already fixed recently.

@aaronmarkham
Copy link
Contributor Author

@szha rebased and passed!

@szha szha merged commit 355de66 into apache:master Jun 29, 2018
XinYao1994 pushed a commit to XinYao1994/incubator-mxnet that referenced this pull request Aug 29, 2018
* fixed path for auto_module_index.js

* nudge flaky ci test
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.

UX for ONNX Documentation is broken
3 participants