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

Devel #1762

Merged
merged 15 commits into from
Sep 29, 2020
Merged

Devel #1762

merged 15 commits into from
Sep 29, 2020

Conversation

dillontsmith
Copy link
Contributor

No description provided.

kmantel and others added 11 commits September 21, 2020 19:49
@kmantel
ParametersTemplate: add __contains__, True if item name or Parameter …
5ab3081 
…exists
@kmantel
StatefulFunctions: set has_initializers True via Parameter, not manually
32ce54f
@kmantel
TransferWithCosts: remove unnecessary setting of some FunctionParameters
3d9c37f
@kmantel
LearningMechanism: make learning_rate a FunctionParameter
82ef53f
@kmantel
OptimizationControlMechanism: add correct setter for control_allocati…
e5504cb 
…on_search_space parameter
@kmantel
components: move class attrs corresponding to parameters
e30880f
@SamKG
llvm/pytorch: Remove bitcasts
daded1b
@SamKG
llvm/pytorch: Remove dead code
49273a0
@SamKG
llvm/pytorch: Use *.pointee.type for builder.alloca calls
ad980fd
@SamKG
Merge pull request #1759 from SamKG/llvm/cleanup/remove_bitcasts
b076129 
llvm cleanup: Remove bitcasts + dead code + change builder alloca
@kmantel
treewide: remove use of dot notation for stateful parameters (#1760)
8708b66
@dillontsmith dillontsmith requested a review from SamKG September 28, 2020 12:59
@lgtm-com
Copy link
Contributor

lgtm-com bot commented Sep 28, 2020

This pull request fixes 1 alert when merging 8708b66 into 46a5489 - view on LGTM.com

fixed alerts:

  • 1 for Variable defined multiple times

@coveralls
Copy link

coveralls commented Sep 28, 2020 •
edited
Loading

Coverage Status

Coverage increased (+0.1%) to 82.683% when pulling 98e5c2c on devel into 46a5489 on master.

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Sep 28, 2020

This pull request fixes 1 alert when merging 98e5c2c into 46a5489 - view on LGTM.com

fixed alerts:

  • 1 for Variable defined multiple times

@dillontsmith dillontsmith merged commit 40726d5 into master Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SamKG SamKG SamKG approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dillontsmith @coveralls @SamKG @kmantel