Skip to content

Update tutorials #490

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
Merged
merged 1 commit into from
Mar 13, 2025
Merged

Update tutorials #490

merged 1 commit into from
Mar 13, 2025

Conversation

@dario-coscia
Copy link
Collaborator

@dario-coscia dario-coscia commented Mar 12, 2025

No description provided.

@dario-coscia dario-coscia added pr-to-fix Label for PR that needs modification v0.2 labels Mar 12, 2025
@dario-coscia dario-coscia force-pushed the update_tutorials branch 3 times, most recently from f0129b0 to d722840 Compare March 12, 2025 17:54
@dario-coscia dario-coscia force-pushed the 0.2 branch 2 times, most recently from 1f54775 to 795e4a4 Compare March 12, 2025 18:40
@dario-coscia dario-coscia force-pushed the update_tutorials branch 4 times, most recently from 8981c79 to bb7471e Compare March 13, 2025 10:22
@dario-coscia dario-coscia requested a review from ndem0 March 13, 2025 10:23
@dario-coscia
Copy link
Collaborator Author

dario-coscia commented Mar 13, 2025 •
edited
Loading

@ndem0

I updated tutorials for 0.2, and modified the pyproject.toml such that now the tutorials extra dependencies can be all installed by python -m pip install "mathlab-pina[tutorials]".

I also updated the trainer because there was a small issue with logging now displaying the test loss (which I found running the tutorials).

Finally, I removed all tutorial.py files, this is because I will create in #479 this workflow (see #493), which automatically generates those file + the HTML files for doc. I think it is clearer to keep this PR only for tutorials.

@FilippoOlivo FilippoOlivo force-pushed the 0.2 branch 3 times, most recently from 7eec394 to 795e4a4 Compare March 13, 2025 13:11
@dario-coscia dario-coscia removed the pr-to-fix Label for PR that needs modification label Mar 13, 2025
@dario-coscia dario-coscia self-assigned this Mar 13, 2025
@dario-coscia dario-coscia merged commit e105318 into 0.2 Mar 13, 2025
16 checks passed
@dario-coscia dario-coscia deleted the update_tutorials branch March 17, 2025 14:13
ndem0 pushed a commit that referenced this pull request Mar 19, 2025
@dario-coscia @ndem0
Update Tutorials 0.2 (#490)
6ce0baf
FilippoOlivo pushed a commit to FilippoOlivo/PINA that referenced this pull request Mar 21, 2025
@dario-coscia @FilippoOlivo
Update Tutorials 0.2 (mathLab#490)
c352d76
FilippoOlivo pushed a commit to FilippoOlivo/PINA that referenced this pull request Apr 17, 2025
@dario-coscia @FilippoOlivo
Update Tutorials 0.2 (mathLab#490)
86222b3
dario-coscia added a commit that referenced this pull request Apr 17, 2025
@dario-coscia
Update Tutorials 0.2 (#490)
278de8e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ndem0 ndem0 Awaiting requested review from ndem0

Assignees

@dario-coscia dario-coscia

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dario-coscia