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

Tests Script #44

Merged
merged 13 commits into from Aug 6, 2021
Merged

Tests Script #44

merged 13 commits into from Aug 6, 2021

Conversation

harshgeek4coder
Copy link
Collaborator

Perform Tests on Package tfwatcher.callbacks

  • Performed Test on tfwatcher.callbacks.epoch -> EpochEnd
  • Performed Test on tfwatcher.callbacks.predict -> PredictEnd
  • Performed Test on tfwatcher.callbacks.test_batch -> TestBatchEnd
  • Performed Test on tfwatcher.callbacks.train_batch -> TrainBatchEnd

Closes #43

Tests Script for Testing the ```tfwatcher``` package
@harshgeek4coder harshgeek4coder added the tests Test out Package stuff label Aug 6, 2021
@harshgeek4coder harshgeek4coder added this to In progress in TF Watcher via automation Aug 6, 2021
@Rishit-dagli Rishit-dagli removed this from In progress in TF Watcher Aug 6, 2021
Copy link
Owner

@Rishit-dagli Rishit-dagli left a comment

Choose a reason for hiding this comment

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

A fundamental flaw with all the tests is that you test .history which is bundled with TensorFlow, the purpose of these tests is to test this package and not TensorFlow's functionalities

setup.py Outdated Show resolved Hide resolved
tests.py Show resolved Hide resolved
tests.py Show resolved Hide resolved
tfwatcher/__init__.py Show resolved Hide resolved
@Rishit-dagli Rishit-dagli linked an issue Aug 6, 2021 that may be closed by this pull request
@harshgeek4coder
Copy link
Collaborator Author

A fundamental flaw with all the tests is that you test .history which is bundled with TensorFlow, the purpose of these tests is to test this package and not TensorFlow's functionalities

actually, my main intention was to get the logs from training so that I could assert and relate some functionalities of the package
that's the only reason why I used history.history for logs retrieval

@Rishit-dagli Rishit-dagli self-requested a review August 6, 2021 14:45
@Rishit-dagli Rishit-dagli merged commit d573ffc into staging Aug 6, 2021
@Rishit-dagli Rishit-dagli deleted the test-script--at-dev branch August 6, 2021 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Test out Package stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Perform Tests on Package
2 participants