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.

Sign up for GitHub

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

Jump to bottom

Test normal class behaviour #386

Closed
PythonFZ opened this issue Sep 14, 2022 · 0 comments · Fixed by #384
Closed

Test normal class behaviour #386

PythonFZ opened this issue Sep 14, 2022 · 0 comments · Fixed by #384
Labels
needs-tests p1-important

Comments

@PythonFZ
Copy link
Member

Write tests that use the ZnTrack Nodes as normal classes without any DVC interaction.
PythonFZ added a commit that referenced this issue Sep 15, 2022
@PythonFZ
fix #386
faccc63
@PythonFZ PythonFZ linked a pull request Sep 15, 2022 that will close this issue
Add Custom Errors #384
Merged
@PythonFZ PythonFZ mentioned this issue Sep 15, 2022
Merged
PythonFZ added a commit that referenced this issue Sep 15, 2022
@PythonFZ
Add Custom Errors (#384)
6f27a1f 
* Add Custom Errors

* update representation

* add load test

* update test

* update error msg

* fix tests

* adjust repr string

* fix test

requires mocking of dvc.yaml because __repr__ is used in a logging statement

* make the hash even more unique

* restrict getitem loading to str|dict

* rename directory

* fix test

* attempt test fix

causes other tests to fail

* possible fix for timeit error

* catch correct exception

* use python3.9 for nbtest / unittest for faster installation

* fix test for Python 3.10

* fix #386

* update test when node is not loaded

* use `dvc stage add` instead of `dvc run`

fixes a test in the docs

* use `dvc stage add` instead of `dvc run`

fixes a test in the docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs-tests p1-important
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Custom Errors zincware/ZnTrack
1 participant
@PythonFZ