Skip to content

Finetuning #134

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

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

Finetuning #134

wants to merge 21 commits into from

Conversation

DavideTisi
Copy link
Collaborator

@DavideTisi DavideTisi commented May 14, 2025

my todo (sofiia):

  • clean up
  • finish example of simple finetuning
  • add some plots or chemiscope
  • add example with nc force learning(pretraining on nc, finetuing on c)
  • verify if recipe generation time can be reduced

Copy link

github-actions bot commented May 14, 2025

Here is the build version of the cookbook in this pull request: documentation.zip, you can view it locally by unzipping documentation.zip and open the index.html with your favorite browser.

@DavideTisi DavideTisi requested a review from cesaremalosso May 19, 2025 13:43
@sofiia-chorna sofiia-chorna marked this pull request as ready for review July 4, 2025 10:02
@ceriottm
Copy link
Contributor

ceriottm commented Jul 6, 2025

Very nice. A few comments for improvement:

  1. would be nice to show training curve (train and validation error vs epoch). also it's not clear if ethanol is computed with a different flavor of DFT, you should indicate that.
  2. It'd be nice (even if it make the model slower) demonstrate the accuracy of a model trained "from scratch" for 1000 epochs. the comparison between training curves would make a nice fig. 1
  3. I'd also show training curves for the conservative fine-tuning, maybe showing the "appended" curve plot as in the paper
  4. another nice thing would be to show in the final parity plot the last point in the NC pre-training in the background with low opacity, so it's clear how much it improved in just 50 epochs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants