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.

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

added keras tuner and updated readme #57

Merged
merged 5 commits into from
Apr 6, 2025
Merged

Conversation

Kanavpreet-Singh
Copy link
Contributor

worked on issue #41

@Cgarg9
Copy link
Collaborator

Cgarg9 commented Apr 2, 2025

@Kanavpreet-Singh well done so far. Small updates -

  1. Lets make a directory for your code inside the model tuning folder.. it should have two files : one for your notebook and one for the assignment
  2. Some code blacks have not been run. please run it completely so that everyone can easily visualise what you aimed to do at each step.

@Kanavpreet-Singh
Copy link
Contributor Author

@Cgarg9 ive tried to implement the suggestions , kindly check and update me .

@Cgarg9
Copy link
Collaborator

Cgarg9 commented Apr 4, 2025

Screenshot 2025-04-04 at 11 14 48 AM from where are we getting these statements? also kindly delete the test.ipynb

@Kanavpreet-Singh
Copy link
Contributor Author

@Cgarg9 , When you call tuner.search(x_train, y_train, epochs=3, validation_split=0.2, callbacks=[early_stopping]), Keras Tuner starts running different trials with different hyperparameters. It automatically logs the results of each trial, including validation accuracy, best accuracy so far, and elapsed time. ive removed test.ipynb file

@Cgarg9
Copy link
Collaborator

Cgarg9 commented Apr 4, 2025

Got it but why is your print statement not present?

@Kanavpreet-Singh
Copy link
Contributor Author

@Cgarg9 sir ive again tried running the cells,
initially it does show the print statement like in the screenshot below
Screenshot 2025-04-05 074630

but afterwards , when trials run , then it only shows info of the particular trial like shown in screenshot below
Screenshot 2025-04-05 075510

At the end , trial 5 ran , so it ended up showing its info

@Cgarg9
Copy link
Collaborator

Cgarg9 commented Apr 6, 2025

Okay got it. I'll look into why this is happening later but we can merge this. Great job.

@Cgarg9 Cgarg9 merged commit 05e9b09 into PEC-CSS:main Apr 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants