-
Notifications
You must be signed in to change notification settings - Fork 67
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
self installing jupyter notebook for eppy tutorial #431
Comments
@jmrocha I have created a new branch i431_notebook. Since you are prototyping, you are welcome to break it in any way want. |
I'm working on updating the main tutorial notebook to fetch the required files online. Also:
|
@jmrocha , |
I’m really sorry to hear you got sick, Santos. Wishing you a speedy recovery. I'll create a work-in-progress PR and include the essential elements (fetching files online and fixing cell errors). We could then take it from there. |
The `/docs/Main_Tutorial.ipynb` notebook now depends on the `eppy` package, along with one `.idd` file and three `.idf` files. Previously, the notebook assumed that the package was already installed and that the files were present in local storage at specific locations. With this commit, the `eppy` package is installed with `pip`, and the required input files are fetched online. The `.idd` file is obtained from the `v23.2.0` tag of the EnergyPlus project and it comes encoded in CP-1252. It is expected to remain stable in the future. The `.idf` files are now fetched from the `master` branch of this repository.
The `/docs/Main_Tutorial.ipynb` notebook now installs the `eppy` package and fetches one `.idd` and three `.idf` files. Previously, the notebook assumed that the package was already installed and that the files were present in local storage at specific locations. - The `eppy` package is installed with `pip`. - The `.idd` file is obtained from the `v23.2.0` tag of the EnergyPlus project. It comes encoded in CP-1252. - The `.idf` files are now fetched from the `master` branch of this repository.
this is a new issue based on
#430 (comment)
Use this issue to prototype the suggestion
The text was updated successfully, but these errors were encountered: