Changed NULL used as a null pointer to nullptr.#11
Open
pengin wants to merge 1 commit into
Open
Conversation
TE-PoornimaBiradar
pushed a commit
that referenced
this pull request
May 28, 2018
…agenet-files Use extractfile in extracting imagenet tar
MasatoIshii-sony
pushed a commit
that referenced
this pull request
Nov 24, 2021
* change layer to calculate shap #3 * visualize shap images #3 * add comment #3 * delete comment out #3 * use variable for num_epochs * change the word Grad-CAM to SHAP #3 * use variable for ratio_num #3 * delete garbage collection #3 * change data_iterator to dataset #3 * rename folder and delete unnecessary files #3 * delete unnecessary files and images #3 * delete unnecessary argument #3 * split into precise functions #3 * add copyright #3 * delete unnecessary DS_Store #3 * fix lint error #3 * change readme #3 * change readme for explainable AI #3 * change image #3 * fix readme for shap #3 * change readme for nnabla-example #3 * fix comment #3 * fix readme for shap #3 * delete DSstore #3 * fix license #3 * delete unnecessary spaces and cells #3 * change the repository where ipynb file is in #3 * change the repository to reference from ghelia to sony #3 * clear the output of the first cell #3 * change readme #3 * deal gloabl variable as an argument #3 * deal error message as an exception handling #3 * delete unnecessary error message #3 * (#3) delete readme and ipynb * (#3) fix readme line break * add 50 images and ipynb file #3 * fix comment #3 * fix layer index #3 * (#3) fix redundunt processing * (#3) fix to designate weight and bias layers with slash in order not to allow mistakes * (#3) zip image to use images by unzipping in notebook * (#3) fix to import shap at the beginning * (#3) delete unnecessary part * (#3) fix explanation * (#3) run with num_samples=100 Co-authored-by: twintrees <kotaro.hayashi@ghelia.com> Co-authored-by: twintrees <70564601+twintrees@users.noreply.github.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Almost all codes use
nullptr, but these two codes don't.If you don't have any special reasons to use
NULL, I think it should be unified.By the way, do you have any CIs to test codes for multiple platforms? I don't have WindowsPC to build and test. I confirmed all tests(excepted for the tests reported #1) have passed only on MacOSX.