It seems like right now you are redefining all your functions in your documentation notebook. Instead, you should just call them from your package. The risk of redefining them like this is that you could easily make a change in one of the two places that doesn't get carried over to the other. Then, depending on the order you run the jupyter cells in, you'll get different outcomes which may be very hard to debug. The only reason to have them redefined like this is during debugging so you can more easily print from within the function or things like that, but even then it's still better to go directly from the package function.
It seems like right now you are redefining all your functions in your documentation notebook. Instead, you should just call them from your package. The risk of redefining them like this is that you could easily make a change in one of the two places that doesn't get carried over to the other. Then, depending on the order you run the jupyter cells in, you'll get different outcomes which may be very hard to debug. The only reason to have them redefined like this is during debugging so you can more easily print from within the function or things like that, but even then it's still better to go directly from the package function.