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

Tolerate Linear layers without bias parameters in profiler #369

Closed
wants to merge 2 commits into from

Conversation

edpizzi
Copy link
Contributor

@edpizzi edpizzi commented Feb 3, 2020

Summary: Handle bias = None as a registered parameter, as used in Lienar(..., bias=False).

Differential Revision: D19700470

Summary:
I've had the profiler break training twice in two weeks, disrupting my workflow. Add a global try/except.

I think users would always want training to continue when model complexity can't be estimated.

Differential Revision: D19700315

fbshipit-source-id: 2e278c1df2c005080c19adea1a9708d463e77c96
Summary: Handle `bias = None` as a registered parameter, as used in `Lienar(..., bias=False)`.

Differential Revision: D19700470

fbshipit-source-id: 1bce47d79cad0e29c5551c36528705cbab2ab6d0
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Feb 3, 2020
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D19700470

facebook-github-bot pushed a commit that referenced this pull request Feb 4, 2020
Summary:
Pull Request resolved: #369

Handle `bias = None` as a registered parameter, as used in `Lienar(..., bias=False)`.

Reviewed By: mannatsingh

Differential Revision: D19700470

fbshipit-source-id: 286c1d40f1ead3d235ed5414aa72c7c8ff959e16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants