Skip to content

doc: add more prominent disclaimer to README.md about development status - #99

Merged
daniel-rdt merged 12 commits into
masterfrom
doc/update-wip-disclaimer
Sep 8, 2025
Merged

doc: add more prominent disclaimer to README.md about development status#99
daniel-rdt merged 12 commits into
masterfrom
doc/update-wip-disclaimer

Conversation

@daniel-rdt

@daniel-rdt daniel-rdt commented Sep 1, 2025

Copy link
Copy Markdown
Member

Changes proposed in this Pull Request

This PR proposes a more prominent wip disclaimer to the project Readme file and improves the limitations and development status section of the readme and documentation.

Disclaimer

image

Improved development status section

image image

Open todos before merging:

  • Update the projects limitations description to include reference to dev status and tyndp methodology
  • Add warning to readthedocs index.rst as well
  • Update dev status

Checklist

  • I tested my contribution locally and it works as intended.
  • Code and workflow changes are sufficiently documented.
  • Changed dependencies are added to envs/environment.yaml.
  • Changes in configuration options are added in config/config.default.yaml.
  • Changes in configuration options are documented in doc/configtables/*.csv.
  • OET license identifier is added to all edited or newly created code files.
  • Sources of newly added data are documented in doc/data_sources.rst.
  • A release note doc/release_notes.rst is added.

@daniel-rdt
daniel-rdt requested a review from tgilon September 1, 2025 14:40
@daniel-rdt daniel-rdt self-assigned this Sep 1, 2025

@tgilon tgilon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also suggest linking to the development status. Currently, the Limitations section refers to PyPSA-Eur limitations. Our current dev status adds up other limitations to reproduce the TYNDP 2024 specifically.

@daniel-rdt

Copy link
Copy Markdown
Member Author

I also suggest linking to the development status. Currently, the Limitations section refers to PyPSA-Eur limitations. Our current dev status adds up other limitations to reproduce the TYNDP 2024 specifically.

I was considering adding the current Open-TYNDP limitations based on the development status to that Limitations page. On the other hand it could make sense to differentiate this into:

  • Limitations: General limitations of the TYNDP and limitations of PyPSA-Eur
  • Development status: Current development status which should include clearly the limitations in terms of incomplete features

What do you think? @tgilon

@daniel-rdt

Copy link
Copy Markdown
Member Author

@tgilon As discussed earlier, I introduced some minor improvements to the limitations section and the development status section of the readme and the documentation. I decided to include a full feature table in the Features in development subsection that also includes completed features as I thought that it looks nicer to see the progress

@tgilon tgilon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! This looks great to me. I've made minor suggestions / comments. I suggest adding an item to the PR template to help keep this updated.

Could you also update the screenshot in the description to improve the doc of the PR? Tks

Comment thread README.md Outdated
Comment thread doc/limitations.rst Outdated
Comment thread doc/limitations.rst Outdated
Comment thread doc/limitations.rst
Comment thread README.md
Comment thread README.md Outdated
Comment thread doc/index.rst Outdated
Comment thread README.md Outdated

@tgilon tgilon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@daniel-rdt
daniel-rdt merged commit f3bb336 into master Sep 8, 2025
5 checks passed
@daniel-rdt
daniel-rdt deleted the doc/update-wip-disclaimer branch September 8, 2025 11:02
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.

2 participants