-
Notifications
You must be signed in to change notification settings - Fork 36
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
gLaSDI update #126
gLaSDI update #126
Conversation
@dylan-copeland @siuwuncheung @kevinhkhuynh where are we with this PR? Do we have enough review for this PR? |
@xiaolong7 Is this PR still in a ready-for-review state, given the recent changes? |
Yes, it is ready for review. |
# plt.savefig(f"./1Dburger_physical_dynamics.png",bbox_inches='tight') | ||
|
||
|
||
# In[ ]: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jupyter Notebook artifacts like these can be removed.
@xiaolong7 I am going to close this PR because we have gLaSDI repo in a separate repo. Is it okay? |
Fine with me!
…On Tue, May 2, 2023, 9:10 PM Youngsoo Choi ***@***.***> wrote:
@xiaolong7 <https://github.com/xiaolong7> I am going to close this PR
because we have gLaSDI repo in a separate repo. Is it okay?
—
Reply to this email directly, view it on GitHub
<#126 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACPYY6UTEFLKPLIAULCX6G3XEHLCNANCNFSM5ZQ6PGQA>
.
You are receiving this because your review was requested.Message ID:
***@***.***>
|
Closing this PR because we have public gLaSDI repo at https://github.com/LLNL/gLaSDI |
Converted notebook to Python scripts
Updated README
Corrected typos in src/error_utils.py