Skip to content

Conversation

@JamieMair
Copy link
Contributor

@JamieMair JamieMair commented Nov 6, 2022

As brought up in this thread, the example from the docs was incomplete, missing methods for the target function and one for selecting the loss for a given epoch (that is my understanding anyway). This was added, along with some spelling and grammar changes as well.

@codecov
Copy link

codecov bot commented Nov 6, 2022

Codecov Report

Base: 73.74% // Head: 73.74% // No change to project coverage 👍

Coverage data is based on head (ada066f) compared to base (8e25936).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #120   +/-   ##
=======================================
  Coverage   73.74%   73.74%           
=======================================
  Files          14       14           
  Lines        2609     2609           
=======================================
  Hits         1924     1924           
  Misses        685      685           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Contributor

@chriselrod chriselrod left a comment

Choose a reason for hiding this comment

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

Thanks!

@chriselrod chriselrod merged commit b817516 into PumasAI:main Nov 6, 2022
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.

2 participants