-
Notifications
You must be signed in to change notification settings - Fork 67
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
WIP: Major revamp #1487
WIP: Major revamp #1487
Conversation
add function to add new sinks
…to logging-improvement
the log from CI shows an upgrade/downgrade for numpy ending with 1.19.5 this may be the cause of the CI fault
This reverts commit 37e81a2.
…to logging-improvement
…to logging-improvement
…ts/scenarios/experiments/base_image_classification.aexp --override model.fit_kwargs.nb_epochs=1 --override execution.mode=native
@davidslater, still a long way to go here, but this at least works with baseline MNIST in native mode. I am going to move on to making it work in Docker and then will work to clean up implementation. Just wanted to get you something to start looking at as I wrap it up. |
@shenshaw26 can you update to black 22.3? (This was part of the 0.15.0 release update). It will make it a lot easier to parse the changes you've made here. |
Will do!! |
I'm having a hard time parsing the changes - it won't even load in my browser. Can you point me to the most relevant files? |
…lower than native and stdout is not as nice but its working
Add |
… are essentially standalone plotting scripts
change bare template strings to quoted
Obsoleted by #1696 which aims for the same goal with a more staged approach than was done here. |
Working to separate Launcher from execution engine.
Started with MNIST Baseline:
armory-launcher -v run ./tests/scenarios/experiments/base_image_classification.aexp --override model.fit_kwargs.nb_epochs=1 --override execution.mode=native
which as of3d80805b13868943a2dc61db89ff45dc1277b661
worked successfully