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

add function to calculate equilibrium points #10

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

Conversation

Wxyalonso14
Copy link

add a new function in modeltools.py to calculate equilibrium points and two test_functions in test_modeltools.py

Copy link
Collaborator

@cknoll cknoll left a comment

Choose a reason for hiding this comment

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

Sorry for commenting not earlier. I had to fix the CI setting first, because I unintentionally broke them long time ago.

I had a first superficial look on this PR. looks good so far. However, it contains many changes to coding style, whitespaces etc. which are unrelated to the actual content. I know that the coding style is suboptimal (excuse: this is for historical reasons), but I prefer not to mix commits with new features with others "just doing cleanup".

Additionally: I renamed the default branch to "main".

Suggestion: Please file a new PR which only contains the necessary changes for the functions/methods which you added. I think the easiest way to do this is by cloning current main branch into a new directory and use copy-paste to insert your changes.

You might also file a second PR which contains changes in coding style. But probably I will do some more refactoring in the future and use an automatic tool like: https://github.com/psf/black so this work would probably be wasted time.

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