Open samlwang2 opened 6 months ago
Thanks for your comment! We appreciate your interest and review :) We intend on keeping the ellipses there, as if i'm not mistaken, this was the way that was done in lecture and worked well for us! In the code we have written, "replace the elipses ...
with a valid 'year_range' for users. "
We made sure all the code works smoothly on all our team member's laptops, feel free to ask or refer to the comments addressing any possible errors if any further issues rise. Thanks again.
Great topic found it super interesting that you were taking the Banana Xanthomonas Wilt disease into consideration! Your git is super organized and I appreciated all the helpful comments but I had a few issues running the master code file.
To fix the ellipsis issue you may need to define year_range as a range of years that your functions can handle. Unsure if that's already defined in another notebook. There are also issues with result if estimate_demand_wrapper is supposed to define result, you may want to double check to ensure that all its dependencies run correctly.