This repository has been archived by the owner on Nov 17, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[MXNET-651] MXNet Model Backwards Compatibility Checker #11626
Merged
Merged
Changes from 2 commits
Commits
Show all changes
61 commits
Select commit
Hold shift + click to select a range
4ee8b21
Added MNIST-MLP-Module-API models to check model save and load_checkp…
piyushghai 118850f
Added LENET with Conv2D operator training file
piyushghai 27863fd
Added LENET with Conv2d operator inference file
piyushghai b3e9774
Added LanguageModelling with RNN training file
piyushghai c141701
Added LamguageModelling with RNN inference file
piyushghai 35cbefb
Added hybridized LENET Gluon Model training file
piyushghai 418f805
Added hybridized LENET gluon model inference file
piyushghai 600efaf
Added license headers
piyushghai d73b9e2
Refactored the model and inference files and extracted out duplicate …
piyushghai 3eeba08
Added runtime function for executing the MBCC files
piyushghai 9c0157c
Added JenkinsFile for MBCC to be run as a nightly job
piyushghai 3d43bcd
Added boto3 install for s3 uploads
piyushghai 4b70e4a
Added README for MBCC
piyushghai 08ad342
Added license header
piyushghai 5d1c3fc
Added more common functions from lm_rnn_gluon_train and inference fil…
piyushghai cfe8dfc
Added scripts for training models on older versions of MXNet
piyushghai 7c41488
Added check for preventing inference script from crashing in case no …
piyushghai 50be5d8
Fixed indentation issue
piyushghai c3c9129
Replaced Penn Tree Bank Dataset with Sherlock Holmes Dataset
piyushghai 3485352
Fixed indentation issue
piyushghai af9b86d
Removed training in models and added smaller models. Now we are simpl…
piyushghai 79cfa46
Updated README
piyushghai 4df779b
Fixed indentation error
piyushghai 04465b0
Fixed indentation error
piyushghai 2d5cf09
Removed code duplication in the training file
piyushghai 7bfdf87
Added comments for runtime_functions script for training files
piyushghai c80ee31
Merged S3 Buckets for storing data and models into one
piyushghai e764d5a
Automated the process to fetch MXNet versions from git tags
piyushghai 05ded05
Added defensive checks for the case where the data might not be found
piyushghai 60c7be0
Fixed issue where we were performing inference on state model files
piyushghai 9d4d099
Replaced print statements with logging ones
piyushghai d08ba5a
Merge branch 'master' into mbcc
piyushghai cebfb26
Removed boto install statements and move them into ubuntu_python docker
piyushghai f7a36eb
Separated training and uploading of models into separate files so tha…
piyushghai 1f63941
Updated comments and README
piyushghai fbaf3e0
Fixed pylint warnings
piyushghai edd6816
Removed the venv for training process
piyushghai 87103d4
Fixed indentation in the MBCC Jenkins file and also separated out tra…
piyushghai eb24e8e
Fixed indendation
piyushghai 3525656
Fixed erroneous single quote
piyushghai 25e7ec7
Added --user flag to check for Jenkins error
piyushghai 00ee6e7
Removed unused methods
piyushghai a3a72b8
Added force flag in the pip command to install mxnet
piyushghai 86e8882
Removed the force-re-install flag
piyushghai ddb672a
Changed exit 1 to exit 0
piyushghai 9e77064
Added quotes around the shell command
piyushghai 69843fb
added packlibs and unpack libs for MXNet builds
piyushghai fae44fe
Changed PythonPath from relative to absolute
piyushghai c099979
Created dedicated bucket with correct permission
marcoabreu ffcc637
Fix for python path in training
piyushghai 7f7f6e3
Merge branch 'mbcc' of https://github.com/piyushghai/incubator-mxnet …
piyushghai 33096c0
Changed bucket name to CI bucket
piyushghai 8a085b5
Added set -ex to the upload shell script
piyushghai 5207ab1
Now raising an exception if no models are found in the S3 bucket
piyushghai 5e30f7a
Added regex to train models script
piyushghai e079d3c
Added check for performing inference only on models trained on same m…
piyushghai ceac705
Added set -ex flags to shell scripts
piyushghai 16d320a
Added multi-version regex checks in training
piyushghai 19495d6
Fixed typo in regex
piyushghai d8fa75d
Now we will train models for all the minor versions for a given major…
piyushghai ca01aa2
Added check for validating current_version
piyushghai File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you run inside a virtualenv, you won't have access to any installed packages. I'd recommend not to use virtualenv here because you're running inside docker anyways. This is also the reason for the CI error that states it could not find boto3.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah I see. Fixed in edd6816