Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed docs links in data classes #1763

Merged

Conversation

kishore-s-15
Copy link
Contributor

Closes #1675

Description: Fixed the links files on user-guide/vision/data-classes directory.

@CLAassistant
Copy link

CLAassistant commented Jul 13, 2022

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@ItayGabbay ItayGabbay left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@kishore-s-15
Copy link
Contributor Author

kishore-s-15 commented Jul 13, 2022

Hi,
I have only changed the document links. Can someone explain to me why my pull request is failing on some tests and the steps to proceed further.

@ItayGabbay ItayGabbay added the documentation modification of the documentation / readme's label Jul 14, 2022
@ItayGabbay
Copy link
Member

@kishore-s-15 some tests which rely on the mnist model are broken because the model has changed. This isn't related to your PR. We'll fix it and then get this PR merged :)

@ItayGabbay ItayGabbay merged commit 6d8ba01 into deepchecks:main Jul 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation modification of the documentation / readme's
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] some links in the data classes implementation tutorial lead to 404
3 participants