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

Surrogate Plugin ML/AI Documentation #1181

Merged
merged 37 commits into from
Dec 13, 2023
Merged

Surrogate Plugin ML/AI Documentation #1181

merged 37 commits into from
Dec 13, 2023

Conversation

bpaul4
Copy link
Contributor

@bpaul4 bpaul4 commented Dec 8, 2023

Summary/Motivation:

Documentation for the machine learning plugin introduced in #1163.

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the copyright and license terms described in the LICENSE.md file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

franflame and others added 30 commits September 5, 2023 14:52
@franflame
Add MVP for ML/AI surrogate plugin
6755609
@franflame
Add copy of data file for testing
a7a1841
@franflame
Formatting changes
39eda3b
@franflame
Resolve pylint error
183549a
@lbianchi-lbl
Initial implementation of GUI tests for surrogate plugins
bc5c7df
@lbianchi-lbl
Skip tests if tensorflow not available
a6061f1
@lbianchi-lbl
Add utility method to export input and output vars directly as dataframe
323520c
@lbianchi-lbl
Add utility method to get currently selected input and output vars data
f48ce3b
@lbianchi-lbl
Add example code for accessing selected input and output vars and data
b0452e1
@lbianchi-lbl
Format
e06bc5a
@lbianchi-lbl
Remove f-string syntax incompatible with Python 3.7
8dd598a
@franflame
Updated to use variables from flowsheet
7c536f5
@lbianchi-lbl
Remove unused and/or redundant local variables for training data
f1e218c
@lbianchi-lbl
Intercept stdout so that training output can be shown in GUI
13ef5f5
@lbianchi-lbl
Change model serialization file format to resolve deprecation warning
6787716
@franflame
Use flowsheet for pytorch_nn.py and scikit_nn.py
7274501
@franflame
Formatting
ab4cdd2
@franflame
Displays pytorch_nn.py output in FOQUS GUI
88dea3a
@franflame
Handle pylint error
ff2eb1f
@lbianchi-lbl
Generalize importorskip using parametrization
9c3fd3a
@lbianchi-lbl
Use different session file for NN training
f60cb84
@lbianchi-lbl
Add variable selection before executing
df8828e
@lbianchi-lbl
Format code
6735cbb
@franflame
Draft description for pytorch_nn.py and scikit_nn.py
dace1e6 
Also fixed spacing in keras_nn.py description
@franflame
Merge remote-tracking branch 'origin/surrogate-plugin-ml-ai' into sur…
91f1917 
…rogate-plugin-ml-ai
@franflame
Added input validation and custom file name option
3680e52
@ksbeattie
Merge branch 'master' into surrogate-plugin-ml-ai
72858da
@franflame
Add ml_ai subdirectory to default path for output file
c7b1a34
@franflame
Merge remote-tracking branch 'origin/surrogate-plugin-ml-ai' into sur…
f2bf3eb 
…rogate-plugin-ml-ai
@bpaul4
Merge branch 'master' into surrogate-plugin-ml-ai
b1531c3
@bpaul4 bpaul4 added Priority:High High Priority Issue or PR ML/AI Machine Learning/Artificial Intelligence labels Dec 8, 2023
@bpaul4 bpaul4 self-assigned this Dec 8, 2023
@codecov Codecov
Copy link

codecov bot commented Dec 8, 2023 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c11cc51) 38.52% compared to head (194aa20) 38.52%.
Report is 1 commits behind head on master.

Additional details and impacted files 
@@           Coverage Diff           @@
##           master    #1181   +/-   ##
=======================================
  Coverage   38.52%   38.52%           
=======================================
  Files         163      163           
  Lines       36880    36880           
  Branches     5995     5995           
=======================================
  Hits        14208    14208           
  Misses      21562    21562           
  Partials     1110     1110

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ksbeattie ksbeattie requested a review from sotorrio1 December 12, 2023 21:02
sotorrio1
sotorrio1 approved these changes Dec 13, 2023
View reviewed changes
Copy link
Member

@sotorrio1 sotorrio1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was able to follow the tutorial with no issues and successfully generated the model file.

@lbianchi-lbl lbianchi-lbl enabled auto-merge (squash) December 13, 2023 01:58
@lbianchi-lbl lbianchi-lbl merged commit f0cf1c3 into CCSI-Toolset:master Dec 13, 2023
14 checks passed
@bpaul4 bpaul4 deleted the surrogate-plugin-ml-ai-docs branch June 3, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sotorrio1 sotorrio1 sotorrio1 approved these changes

Assignees

@bpaul4 bpaul4

Labels
ML/AI Machine Learning/Artificial Intelligence Priority:High High Priority Issue or PR
Projects
None yet
Milestone
EY23.4.1.M: Expand existing Surrogate...
Development

Successfully merging this pull request may close these issues.
5 participants
@bpaul4 @sotorrio1 @lbianchi-lbl @franflame @ksbeattie