Skip to content

Conversation

@enryH
Copy link
Collaborator

@enryH enryH commented Jan 17, 2025

No description provided.

@enryH enryH requested a review from sayalaruano January 17, 2025 14:42
@enryH
Copy link
Collaborator Author

enryH commented Jan 17, 2025

To use the config file directly provided with the repo it is still necessary to got into the docs folder, so from root the following code does not work: vuegen -c docs/example_data/MicW2Graph/report_config_micw2graph.yaml --report_type streamlit

@sayalaruano
Copy link
Collaborator

I fixed the component paths of ´report_config_micw2graph.yaml´ by appending "docs/" and vuegen is running locally from the root. But, now the tests are not passing bc of this change.

@enryH
Copy link
Collaborator Author

enryH commented Jan 21, 2025

I fixed the component paths of ´report_config_micw2graph.yaml´ by appending "docs/" and vuegen is running locally from the root. But, now the tests are not passing bc of this change.

The workflow for tests I would not touch. It's later also Sphinx specific issues (the results need to be in the docs folder). I will revert you changes, also in the light of the changes in #44

@enryH
Copy link
Collaborator Author

enryH commented Jan 21, 2025

So the issue is with the example notebook. If I run vuegen_demo.ipynb through sphinx the working directory will be docs. Therefore I checked that execution for that notebooked worked as expected.

RuntimeError: An error occurred while reading the network file: The file at docs/example_data/MicW2Graph/2_Microbial_Association_Networks/1_Network_visualization1/1_man_example.graphml was not found or cannot be accessed.

What we could do is change the directory execution of the workflow - if you agree!

@enryH enryH merged commit 48de138 into main Jan 30, 2025
11 checks passed
@enryH enryH deleted the pkg_restructuring branch January 30, 2025 13:29
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.

3 participants