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

first pass at README review from fresh eyes #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

paddymul
Copy link

No description provided.

@@ -1,7 +1,21 @@
# Poietic Flows

Systems Dynamics Modelling and simulation toolkit based on the
[Stock and Flow](https://en.wikipedia.org/wiki/Stock_and_flow) methodology.
[Stock and Flow](https://en.wikipedia.org/wiki/Stock_and_flow) methodology .
Copy link
Author

Choose a reason for hiding this comment

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

Describing what this repo does with a link to the wikipedia description is a really nice start. I read so many repos and have to get deep into the README before I even have an idea about the problem it is trying to solve.

That said, you have 3 links to the same wikipedia article in the README

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for pointing that out. Happened by just copy-pasting snippets from other notes/docs. Will correct that.

Copy link
Contributor

@Stiivi Stiivi left a comment

Choose a reason for hiding this comment

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

Thank you for the feedback, will consider it, reword it and add additional information.

@@ -1,7 +1,21 @@
# Poietic Flows

Systems Dynamics Modelling and simulation toolkit based on the
[Stock and Flow](https://en.wikipedia.org/wiki/Stock_and_flow) methodology.
[Stock and Flow](https://en.wikipedia.org/wiki/Stock_and_flow) methodology .
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for pointing that out. Happened by just copy-pasting snippets from other notes/docs. Will correct that.

## Who it's for

I wrote this when solving problem ???. It would be applicable to economists, and X and X.

Copy link
Contributor

Choose a reason for hiding this comment

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

The software is not yet ready for the intended target audience. For now, it is just "for the curious". I'll write a longer post about the larger vision, including the problems I am trying to address in the domain of SD modelling, once I wrap-up the preview.

@@ -35,6 +49,8 @@ Planned:

## Installation

Poietec runs on Mac OS X, ?? and ??.
Copy link
Contributor

Choose a reason for hiding this comment

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

Good point. Will note that. Had to do some fixes to make it work again on Linux (there are some methods that are deprecated on MacOS yet not available on Linux).

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