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

Docs: Remove aiida.manage.configuration.load_documentation_profile #1000

Merged
merged 1 commit into from
Dec 22, 2023

Conversation

sphuber
Copy link
Contributor

@sphuber sphuber commented Dec 22, 2023

This was necessary when Django backends were still supported, but since
its removal in AiiDA v2.0 just loading the ORM no longer actually needs
to connect to a database. In principle, the documentation no longer
needs a profile whatsoever, except for aiida-quantumespresso the
AutoAPI still needs it because in loading the specifications of certain
Process classes, a profile is loaded. So the docs conf.py now loads
an empty dummy profile for this purpose.

@sphuber sphuber force-pushed the fix/docs branch 2 times, most recently from bc90583 to 9344b61 Compare December 22, 2023 14:44
This was necessary when Django backends were still supported, but since
its removal in AiiDA v2.0 just loading the ORM no longer actually needs
to connect to a database. In principle, the documentation no longer
needs a profile whatsoever, except for `aiida-quantumespresso` the
AutoAPI still needs it because in loading the specifications of certain
`Process` classes, a profile is loaded. So the docs `conf.py` now loads
an empty dummy profile for this purpose.
@sphuber sphuber merged commit f1d547c into main Dec 22, 2023
13 checks passed
@sphuber sphuber deleted the fix/docs branch December 22, 2023 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant