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

Adding short version of OSeMOSYS PULP and udpates to the long code #19

Conversation

ShravanKumar23
Copy link
Collaborator

No description provided.

@ShravanKumar23 ShravanKumar23 requested a review from willu47 June 9, 2023 10:15
@willu47 willu47 self-assigned this Jun 9, 2023
# FUNCTIONS
# ----------------------------------------------------------------------------------------------------------------------

def createParameter(_df, _name):
Copy link
Member

Choose a reason for hiding this comment

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

These utility functions are duplicated over the various versions of the code. I suggest moving these into a utility module e.g. utils.py and then importing them.

Copy link
Member

Choose a reason for hiding this comment

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

Should this be part of the pull request? If not, please remove. If this is a to-do list, consider adding these as individual issues on Github.

Copy link
Member

Choose a reason for hiding this comment

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

Should this be part of the pull request as it looks like a result file?


OSeMOSYS-PuLP-HP

This is the high performance (HP) version of OSeMOSYS-PuLP
Copy link
Member

Choose a reason for hiding this comment

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

Update this comment.

@@ -0,0 +1,1013 @@
# !/usr/bin/env python3
# -*- coding: utf-8 -*-
# Author: Dennis Dreier, Copyright 2020
Copy link
Member

Choose a reason for hiding this comment

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

Update author with Shravan


Please cite this software by using the following reference of the original scientific article:

Dennis Dreier, Mark Howells, OSeMOSYS-PuLP: A Stochastic Modeling Framework for Long-Term Energy Systems Modeling.
Copy link
Member

Choose a reason for hiding this comment

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

Add citation as a citation.cff file in the repository.


"""

path_to_cplex = r'C:/Program Files/IBM/ILOG/CPLEX_Studio1262/cplex/bin/x64_win64/cplex.exe'
Copy link
Member

Choose a reason for hiding this comment

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

This should be in a user-defined config file somewhere, not hard-coded.

@willu47 willu47 linked an issue Jun 12, 2023 that may be closed by this pull request
@willu47 willu47 assigned ShravanKumar23 and unassigned willu47 Jun 20, 2023
@ShravanKumar23 ShravanKumar23 deleted the Updated-storage-equations-and-other-stuff branch November 26, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Add OSeMOSYS PULP Short
2 participants