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

Add support for multi-layer perceptron estimators #40

Merged
merged 1 commit into from Jan 29, 2022

Conversation

iamDecode
Copy link
Owner

This PR adds initial support for neural network models. Non-fully connected networks are supported using fully connected multi-layer perceptrons, that simply have a bunch of zero-weight neuron connections for the ones not described in the PMML document.

There is no standard way of dealing with categorical variables in neural networks, hence I have left categorical support out of this PR for brevity. It is currently up to the user to encode categorical variables first.

@codecov
Copy link

codecov bot commented Jan 28, 2022

Codecov Report

Merging #40 (3c5ccc2) into master (45d4b2c) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   99.62%   99.65%   +0.02%     
==========================================
  Files          19       22       +3     
  Lines        1062     1146      +84     
==========================================
+ Hits         1058     1142      +84     
  Misses          4        4              
Impacted Files Coverage Δ
sklearn_pmml_model/neural_network/__init__.py 100.00% <100.00%> (ø)
sklearn_pmml_model/neural_network/_base.py 100.00% <100.00%> (ø)
sklearn_pmml_model/neural_network/_classes.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 45d4b2c...3c5ccc2. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented Jan 28, 2022

This pull request introduces 2 alerts when merging f7fb5d0 into 45d4b2c - view on LGTM.com

new alerts:

  • 2 for Conflicting attributes in base classes

@iamDecode iamDecode force-pushed the decode/neural-network branch 2 times, most recently from 4ccea04 to 582e75e Compare January 28, 2022 13:50
@iamDecode iamDecode merged commit 02b7a0b into master Jan 29, 2022
@iamDecode iamDecode deleted the decode/neural-network branch January 29, 2022 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant