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

Fix broken URL for Face Recognition #15683

Merged
merged 1 commit into from
Jul 30, 2019
Merged

Fix broken URL for Face Recognition #15683

merged 1 commit into from
Jul 30, 2019

Conversation

healess
Copy link
Contributor

@healess healess commented Jul 29, 2019

Description

This PR creates error pages for face recognition (ArcFace)
page not found error(404)

Checklist

Essentials

Please feel free to remove inapplicable items for your PR.

  • Changes are complete (i.e. I finished coding on this PR)
  • To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

  • Fix bunch of broken links.

@healess
Fix url (404 error)
d609d84 
Fix page url for face recognition - page not found error(404)
@healess healess requested a review from szha as a code owner July 29, 2019 05:04
wkcn
wkcn approved these changes Jul 30, 2019
View reviewed changes
Copy link
Member

@wkcn wkcn left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for the fix!

@healess healess closed this Jul 30, 2019
@healess healess reopened this Jul 30, 2019
@eric-haibin-lin eric-haibin-lin merged commit 93708eb into apache:master Jul 30, 2019
anirudhacharya pushed a commit to anirudhacharya/mxnet that referenced this pull request Aug 20, 2019
@healess
Fix url (404 error) (apache#15683)
12c5ede 
Fix page url for face recognition - page not found error(404)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@wkcn wkcn wkcn approved these changes

@szha szha Awaiting requested review from szha szha is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@healess @wkcn @eric-haibin-lin