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

SorrTask1018_clean_up_notebooks #1019

Merged
merged 3 commits into from
Jun 3, 2024
Merged

Conversation

tkpratardan
Copy link
Collaborator

Did some cleaning up in the notebooks to make it consistent with our coding style.

  • Added utils.py to have all the utility functions in one place.
  • Made changes to sota_baseline_glm.ipyn

@gpsaggese @sonaalKant

@tkpratardan tkpratardan self-assigned this May 31, 2024
@tkpratardan tkpratardan linked an issue May 31, 2024 that may be closed by this pull request
@tkpratardan tkpratardan added the PR_for_reviewers The PR needs to be reviewed by Team Leaders label May 31, 2024
Copy link
Contributor

@gpsaggese gpsaggese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's merge and then we continue adding code / clean up

@gpsaggese gpsaggese merged commit a0c8bdb into master Jun 3, 2024
1 check passed
@gpsaggese gpsaggese deleted the SorrTask1018_clean_up_notebooks branch June 3, 2024 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR_for_reviewers The PR needs to be reviewed by Team Leaders
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clean up notebooks
3 participants