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

Fix/example section in the readme is still refering to the oemof example repository #954

Conversation

p-snft
Copy link
Member

@p-snft p-snft commented Jul 6, 2023

Example section in the README was still referring to the oemof-example repository. This PR fixes that. Secondly, the quality of the integration of the examples into the documentation is improved.

p-snft and others added 6 commits July 6, 2023 09:17
Examples for solph are no longer in a separate repository.
It's no longer impossible to have both packages, oemof and oemof.solph,
installed. So the documentation should no longer claim it to be impossible.
@p-snft
Copy link
Member Author

p-snft commented Jul 30, 2023

Looking at oemof/oemof-examples#84, we should really have this fix done.

@fwitte
Copy link
Member

fwitte commented Jul 30, 2023

Maybe also discontinue the examples Repo? Or refer to the correct locations from there?

@p-snft p-snft added this to the v0.5.1 milestone Aug 8, 2023
The example had no __name__ == "__main__" protector,
so it was run on import. Ths lead to a failure in building
docs on systems without cbc installed.
There sems to be an issue with newer versions in combination with the
rdt theme.
@p-snft
Copy link
Member Author

p-snft commented Aug 11, 2023

Maybe also discontinue the examples Repo? Or refer to the correct locations from there?

I would also discontinue it. In the current form, it has no added value.

@p-snft
Copy link
Member Author

p-snft commented Aug 11, 2023

I (temporarily) activated this branch on RTD so that you can see how the examples render in the solph documentation. I really like it and consider this PR done.

@p-snft p-snft marked this pull request as ready for review August 11, 2023 13:57
@p-snft p-snft self-assigned this Aug 11, 2023
@fwitte
Copy link
Member

fwitte commented Aug 11, 2023

I can have a look on like Sunday if it is sufficient

@p-snft
Copy link
Member Author

p-snft commented Aug 11, 2023

I can have a look on like Sunday if it is sufficient

Sure. My plan is to create the release middle of next week.

@p-snft
Copy link
Member Author

p-snft commented Aug 13, 2023

I just realised that there are still links to the example repository, e.g. at https://oemof-solph.readthedocs.io/en/fix-example-section-in-the-readme-is-still-refering-to-the-oemof-example-repository/usage.html#using-the-excel-csv-reader.

Fixed. The links now point to the examples in the same documentation.

@p-snft p-snft merged commit 1952bf8 into dev Aug 15, 2023
14 checks passed
@p-snft p-snft deleted the fix/example-section-in-the-readme-is-still-refering-to-the-oemof-example-repository branch August 15, 2023 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Example section in the README is still refering to the oemof-example repository
2 participants