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.

Sign up for GitHub

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

Jump to bottom

Tensorflow 2.x backend: multiple outputs extension of DeepONet #1533

Merged
merged 19 commits into from
Dec 22, 2023
Merged

Tensorflow 2.x backend: multiple outputs extension of DeepONet #1533

merged 19 commits into from
Dec 22, 2023

Conversation

vl-dud
Copy link
Contributor

@vl-dud vl-dud commented Oct 23, 2023 •
edited
Loading

This pull request suggests an enhancement to DeepONet, addressing its current limitation of producing only a scalar output, which restricts its applicability in various tasks. The functionality of handling multiple outputs has been added to both the DeepONet and DeepONetCartesianProd classes.
This is an implementation for TensorFlow 2.x backend.

@vl-dud vl-dud mentioned this pull request Oct 23, 2023
Closed
@lululxvi
Copy link
Owner

lululxvi commented Nov 5, 2023

Use a new PR for the example.

This was referenced Nov 7, 2023
Open
Closed
@vl-dud
Copy link
Contributor Author

vl-dud commented Nov 7, 2023

Use a new PR for the example.

See #1550

@lululxvi
Copy link
Owner

lululxvi commented Dec 1, 2023

Please resolve the conflicts.

lululxvi
lululxvi reviewed Dec 1, 2023
View reviewed changes
deepxde/data/triple.py Outdated Show resolved Hide resolved
@lululxvi
Copy link
Owner

lululxvi commented Dec 2, 2023

have you tested the code for data-drive and physics-informed cases?

@vl-dud
Copy link
Contributor Author

vl-dud commented Dec 12, 2023

Yes, I tested both cases. But in real research I use Tensorflow 1.x and don't work much with this branch.

@lululxvi
Copy link
Owner

Revise the code using the TensorFlow v1 example.

@lululxvi lululxvi merged commit d42a6ca into lululxvi:master Dec 22, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lululxvi lululxvi lululxvi left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vl-dud @lululxvi