python-novice-inflammation icon indicating copy to clipboard operation
python-novice-inflammation copied to clipboard

Add explicit parameters for numpy.delete

Open ndporter opened this issue 3 years ago • 1 comments

The alternative solution of using numpy.delete() to remove a column from an array is helpful, but the meaning of the arguments is not obvious without explicitly including the names. Even then, "obj" is non-obvious, so I also added a note in the solution description reminding them to use help files if necessary.

Instructions

Thanks for contributing! :heart:

If this contribution is for instructor training, please email the link to this contribution to [email protected] so we can record your progress. You've completed your contribution step for instructor checkout by submitting this contribution!

Keep in mind that lesson maintainers are volunteers and it may take them some time to respond to your contribution. Although not all contributions can be incorporated into the lesson materials, we appreciate your time and effort to improve the curriculum. If you have any questions about the lesson maintenance process or would like to volunteer your time as a contribution reviewer, please contact The Carpentries Team at [email protected].

You may delete these instructions from your comment.

- The Carpentries

ndporter avatar Jan 05 '22 16:01 ndporter

Edit: I created a new issue about my original comment. Instead, I think the pull request is a valuable improvement although I am not really convinced that showing numpy.delete is beneficial at all.

chillenzer avatar Sep 15 '22 11:09 chillenzer