Skip to content

updates to run with mac/linux#75

Open
ktehranchi wants to merge 9 commits into
pencleanenergy:masterfrom
ktehranchi:master
Open

updates to run with mac/linux#75
ktehranchi wants to merge 9 commits into
pencleanenergy:masterfrom
ktehranchi:master

Conversation

@ktehranchi
Copy link
Copy Markdown

@ktehranchi ktehranchi commented Oct 16, 2023

Contribution License Agreement

By making a contribution to this repository, I agree to the terms of the Contribution License Agreement.
Please select one of the following:

  • I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
  • I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. Name of employer: [enter name of employer]

Description of Pull Request

  • Specify environment versions correctly.
  • Updates to run_scenarios.ipynb and generate_reports to enable running with mac/linux os
  • New cells in run_scenarios to help with cleaning model_workspaces
  • The summary reports still fails on mac, but i made changes so user can manually run the notebook without throwing an error. The root of the issue is that with macOS using the os.system() to call jupyter nbconvert --execute will not recognize the conda environment you have enabled, and instead default to the root environment.... I couldn't figure out a solution to this.

@ktehranchi ktehranchi marked this pull request as draft October 16, 2023 18:34
@ktehranchi ktehranchi marked this pull request as ready for review October 16, 2023 21:26
@ktehranchi
Copy link
Copy Markdown
Author

Like I mentioned in my email. This hardcodes cbc as the solver and ignore the other solver options specified in the xlsx for model inputs.

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.

1 participant