Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

First WARG Commit #181

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

noah-yacowar
Copy link

The first screenshot includes console output of accuracies. The training dataset accuracy was just under 96% accurate, and same with the testing dataset.

In the second picture the second graph includes both the training data set and validation dataset losses.

Please let me know what else is needed!



# Determines whether the machine is Cuda enabled, allowing GPU access
def get_default_device():
Copy link

Choose a reason for hiding this comment

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

Nit: It is good practice to also specify what the return type will be for the function. It helps make the code more readable.

Copy link

@mgupta27 mgupta27 left a comment

Choose a reason for hiding this comment

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

LGTM!

Congratulations and welcome to the Autonomy sub-team at WARG. For next steps please refer to https://uwarg-docs.atlassian.net/wiki/spaces/AD/pages/1697349760/New+Member+Onboarding. You will notice there is also a subpage for new Autonomy members as well. Please take a look at that as well.

Meetings are TBD but will be announced soon in the Discord. Can't wait to see you there!

@@ -1,17 +1,271 @@
"""
Copy link

Choose a reason for hiding this comment

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

Note: Regarding naming convention for PR's, it's good practice to name the PR relevant to the task you are working on. A PR is different from a commit so "First WARG Commit" isn't a great a PR name (but that is ok because we are here to learn and it is totally fine to make mistakes. I've made some myself when I first joined 😄). If you would like more help with understanding Git feel free to reach out.

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

Successfully merging this pull request may close these issues.

2 participants