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

mlflow module not showing up on the documentation page #932

Closed
wd60622 opened this issue Aug 14, 2024 · 5 comments · Fixed by #943
Closed

mlflow module not showing up on the documentation page #932

wd60622 opened this issue Aug 14, 2024 · 5 comments · Fixed by #943
Labels
docs Improvements or additions to documentation mlflow

Comments

@wd60622
Copy link
Contributor

wd60622 commented Aug 14, 2024

The mlflow module has not been showing up in the documentation even with the merge to the main branch. View the documentation here.

This would want to be added such that it displays.

@wd60622 wd60622 added docs Improvements or additions to documentation mlflow labels Aug 14, 2024
@wd60622
Copy link
Contributor Author

wd60622 commented Aug 17, 2024

Would @OriolAbril understand why this is not showing up?

@OriolAbril
Copy link
Collaborator

sphinx autodoc imports the objects it documents, and it is not being able to import it given mlflow is needed but it is not installed in the docs env.

There are two options:

I'd probably try the mock option and check what happens with the docs of autolog, if the decorator modifies the docstrings/methods dynamically it might end up looking weird when that doesn't happen.

@wd60622
Copy link
Contributor Author

wd60622 commented Aug 18, 2024

sphinx autodoc imports the objects it documents, and it is not being able to import it given mlflow is needed but it is not installed in the docs env.

There are two options:

I'd probably try the mock option and check what happens with the docs of autolog, if the decorator modifies the docstrings/methods dynamically it might end up looking weird when that doesn't happen.

You are a lifesaver! Will try out these suggestions

@wd60622
Copy link
Contributor Author

wd60622 commented Aug 18, 2024

There might be a conflict since the package name is mlflow and our module is also same name. Giving the auto_mock_imports a try first

@wd60622
Copy link
Contributor Author

wd60622 commented Aug 18, 2024

Thanks @OriolAbril !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation mlflow
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants