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

rethink the logging code from the LoggedLitModule #26

Closed

charlesfrye opened this issue Jun 2, 2021 · 0 comments

Labels

lightning

Contributor

charlesfrye commented Jun 2, 2021

This was written when I had half as much experience with Lighting as I have now, and before the most recent integration.

I should rethink it, with emphasis on the following:

moving material into callbacks
cutting down on the magic parameter inference
improve robustness (see improve the robustness of the utils #25)
reorganize and make hierarchical (see Consider creating a nn utilities file #20)
writing some tests, including e2e tests that write runs to wandb and pull metrics

charlesfrye added the lightning label

charlesfrye mentioned this issue

Deduplicate projects #27

Open

charlesfrye closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment