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.

Sign up for GitHub

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

Jump to bottom

Fix Doxygen documentation #260

Closed
dinkelbachjan opened this issue Oct 26, 2023 · 1 comment · Fixed by #264
Closed

Fix Doxygen documentation #260

dinkelbachjan opened this issue Oct 26, 2023 · 1 comment · Fixed by #264
Assignees
@leonardocarreras

Comments

@dinkelbachjan
Copy link
Contributor

The doxygen documentation is broken:
https://sogno.energy/dpsim/doxygen/index.html
leonardocarreras added a commit that referenced this issue Nov 10, 2023
@leonardocarreras
Fixes the doxygen documentation input line
9f9c80d 
With the new structure in PR #94, that is:
* dpsim
* dpsim-models
* dpsim-villas
each one has a src and include folders. This breaks the INPUT argument in Doxygen.

This commit fixes #260.

Signed-off-by: Leonardo Carreras <leonardo.carreras@eonerc.rwth-aachen.de>
@leonardocarreras leonardocarreras mentioned this issue Nov 10, 2023
Merged
@leonardocarreras leonardocarreras self-assigned this Nov 10, 2023
@leonardocarreras
Copy link
Collaborator

leonardocarreras commented Nov 10, 2023 •
edited
Loading

image

In local is working correctly with the submitted PR.

Related but not part of this problem: the actions worker is using -j$(nproc) as argument in the compilation... Should this be corrected in a new issue?

dpsim/.github/workflows/documentation.yaml

Lines 24 to 27 in d2cfcda

- name: Make reference docs
run: |
make -j$(nproc) -C build docs
make -j$(nproc) -C build docs_cxx

m-mirz pushed a commit that referenced this issue Mar 1, 2024
@leonardocarreras @m-mirz
Fixes the doxygen documentation input line
2f92013 
With the new structure in PR #94, that is:
* dpsim
* dpsim-models
* dpsim-villas
each one has a src and include folders. This breaks the INPUT argument in Doxygen.

This commit fixes #260.

Signed-off-by: Leonardo Carreras <leonardo.carreras@eonerc.rwth-aachen.de>
m-mirz added a commit that referenced this issue Mar 1, 2024
@m-mirz
Fixes to the doxygen documentation INPUT line (#264)
29c0145 
With the new structure in PR #94, that is:
* dpsim
* dpsim-models
* dpsim-villas each one has a src and include folders. This breaks the
INPUT argument in Doxygen.

This commit fixes this and resolves #260.
martinmoraga pushed a commit that referenced this issue Mar 22, 2024
@leonardocarreras @martinmoraga
Fixes the doxygen documentation input line
abfd660 
With the new structure in PR #94, that is:
* dpsim
* dpsim-models
* dpsim-villas
each one has a src and include folders. This breaks the INPUT argument in Doxygen.

This commit fixes #260.

Signed-off-by: Leonardo Carreras <leonardo.carreras@eonerc.rwth-aachen.de>
martinmoraga pushed a commit that referenced this issue Mar 22, 2024
@leonardocarreras @martinmoraga
Fixes the doxygen documentation input line
8d07917 
With the new structure in PR #94, that is:
* dpsim
* dpsim-models
* dpsim-villas
each one has a src and include folders. This breaks the INPUT argument in Doxygen.

This commit fixes #260.

Signed-off-by: Leonardo Carreras <leonardo.carreras@eonerc.rwth-aachen.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leonardocarreras leonardocarreras

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes to the doxygen documentation INPUT line sogno-platform/dpsim
2 participants
@dinkelbachjan @leonardocarreras