Keras file format updates #128
Conversation
| @@ -443,16 +444,30 @@ | |||
| "cell_type": "markdown", | |||
| "metadata": {}, | |||
| "source": [ | |||
| "Now we have our property package ready for being used in the flowsheet for optimization. We shall see that in the next part of this tutorial, [flowsheet_optimization](./flowsheet_optimization_usr.ipynb). To learn in detail about making a custom property package, one should go through [Property Package Example](../../../properties/custom/custom_physical_property_packages_usr.ipynb). " | |||
There was a problem hiding this comment.
@bpaul4 was this change required because of a specific error? Usually I'd expect (but haven't actually checked for Jupyter/JupyterLab) that Unix-style paths with / are accepted on Windows as well, so I'd be curious to know if there might be something that warranted this change.
There was a problem hiding this comment.
That's very odd. The double slash change doesn't exist on my local branch; there's another change in "_doc" version of this notebook from ".ipynb" to ".md" that I am sure I did not make. Not sure how this happened, as the other versions should all be identical to the source notebook, correct?
There was a problem hiding this comment.
I agree, that's very strange. I'll try to look into it, starting from attempting to reproduce this locally.
requirements-dev.txt
Outdated
| # idaes-pse @ git+https://github.com/IDAES/idaes-pse@main | ||
| # FIXME: revert back to IDAES/idaes-pse@main before merging | ||
| idaes-pse @ git+https://github.com/IDAES/idaes-pse@refs/pull/1401/merge |
There was a problem hiding this comment.
FIXME: revert this before merging
|
We should proceed as follows:
|
|
The 3.8 tests appear stuck, and everything else is passing. @lbianchi-lbl is the plan to remove 3.8 support in this repository, and remove the 3.8 tests? FYI per @andrewlee94 we probably need to merge this PR first, and then 1401 in idaes-pse. |
Good point @bpaul4, yes, let me open a PR for that. EDIT Actually, on second thought, I'll just update and merge this PR since I think the changes in this PR are needed to make tests pass with Python 3.12. |
Fixes #88.
Dependent on IDAES/idaes-pse#1401 for tests to pass.
Legal Acknowledgement
By contributing to this software project, I agree to the following terms and conditions for my contribution:
📚 Documentation preview 📚: https://idaes-examples--128.org.readthedocs.build/en/128/